Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Revert "(MODULES-6408) - Fix dirmngr install failing" #808

Merged
merged 1 commit into from
Oct 1, 2018

Conversation

eimlav
Copy link

@eimlav eimlav commented Oct 1, 2018

Reverts #801. Reverting as this is causing failures in Puppet DB.

@pmcmaw pmcmaw merged commit 68168a1 into master Oct 1, 2018
@eimlav eimlav added the bugfix label Oct 1, 2018
@pmcmaw pmcmaw removed the maintenance label Oct 1, 2018
@rvdh
Copy link

rvdh commented Oct 2, 2018

This revert actually breaks the APT module for me:

Info: Using configured environment 'production'
Info: Retrieving pluginfacts
Info: Retrieving plugin
Info: Loading facts
Info: Caching catalog for client1.vagrant.lan
Info: Applying configuration version '1538485599'
Error: Failed to apply catalog: Found 1 dependency cycle:
(Package[dirmngr] => Class[Apt] => Package[dirmngr])
Try the '--graph' option and opening the resulting '.dot' file in OmniGraffle or GraphViz
root@client1:~#

After commenting the ensure_packages line:

  # manage pins if present
  if $pins {
    create_resources('apt::pin', $pins)
  }

  # required for adding GPG keys on Debian 9 (and derivatives)
  #ensure_packages(['dirmngr'])
}

The manifest works again:

root@client1:~# puppet agent -t
Info: Using configured environment 'production'
Info: Retrieving pluginfacts
Info: Retrieving plugin
Info: Loading facts
Info: Caching catalog for client1.vagrant.lan
Info: Applying configuration version '1538485760'
Notice: /Stage[main]/Avisi_restore/Service[restic-restore]/ensure: ensure changed 'stopped' to 'running'
...
Notice: /Stage[main]/Avisi_localpostfix/Postfix::Hash[/etc/postfix/mydestinations]/Postfix::Map[/etc/postfix/mydestinations]/File[postfix map /etc/postfix/mydestinations.db]/mode: mode changed '0600' to '0644'
Notice: Applied catalog in 168.27 seconds```

@eimlav
Copy link
Author

eimlav commented Oct 2, 2018

Hey @rvdh . Would it be possible for you to open a ticket over at https://tickets.puppetlabs.com along with the details of your environment and your error so that we can document the issue? Thanks!

@david22swan david22swan deleted the revert-801-modules6408 branch October 16, 2018 13:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants