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

(PUP-11939) Don't manage firewall in smoke test #2389

Merged
merged 1 commit into from
Sep 13, 2023

Conversation

joshcooper
Copy link
Contributor

@joshcooper joshcooper commented Sep 11, 2023

The latest release of puppetlabs-puppetdb depends on a version of firewall that doesn't support puppet8.

I intentionally didn't target 7.x, because it's only an issue with puppet8 and the puppetdb/firewall modules.

There are two 2 smoke tests described in https://github.com/puppetlabs/puppet-agent/tree/main/ext/smoke#overview

  • install from packages
    ./packages/run-smoke-test.sh suave-easement.delivery.puppetlabs.net retentive-creed.delivery.puppetlabs.net 8.2.0 8.2.1 8.1.0
  • install from shared repository
    ./repos/run-smoke-test.sh coy-pageantry.delivery.puppetlabs.net tart-breeze.delivery.puppetlabs.net astral-roomful.delivery.puppetlabs.net faulty-repeater.delivery.puppetlabs.net 8.2.0 8.2.1 8.1.0

@joshcooper joshcooper requested a review from a team as a code owner September 11, 2023 20:55
Copy link
Contributor

@tvpartytonight tvpartytonight left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested with both smoke tests, LGTM.

The latest release of puppetlabs-puppetdb depends on a version of firewall that
doesn't support puppet8.
@joshcooper joshcooper merged commit d9c0254 into puppetlabs:main Sep 13, 2023
3 checks passed
@joshcooper joshcooper deleted the repo_smoke_5713 branch September 13, 2023 23:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants