Skip to content

[BUG] Issues with repo install.sls in v1.0.0 #258

@hkbakke

Description

@hkbakke

https://github.com/saltstack-formulas/docker-formula/blob/master/docker/software/package/repo/install.sls has a couple of issues:

  1. refresh is misspelled as 'refesh'
  2. Onlyif seems to not be used correctly. As I have understood it should either specify a command or a module, not be fed a dictionary raw from pillar. I get the following warning when executing with a custom repo:
[WARNING ] no `fun` argument in onlyif: OrderedDict([('comments', ['installed by salt']), ('enabled', 1), ('gpgcheck', 1), ('name', 'deb [arch=amd64] http://mirror.example.com/download.docker.com/linux/debian buster stable'), ('file', '/etc/apt/sources.list.d/docker.list'), ('key_url', 'http://mirror.example.com/keys/docker.gpg.key')])                                       

Pillar:

docker:
  pkg:
    docker:
      use_upstream: package
      repo:
        name: deb [arch=amd64] http://mirror.example.com/download.docker.com/linux/debian {{ grains['oscodename'] }} stable
        key_url: http://mirror.example.com/keys/docker.gpg.key

Metadata

Metadata

Assignees

No one assigned

    Labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions