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

(PE-38685) Beaker 6 upgrade #281

Merged
merged 2 commits into from
Jun 21, 2024
Merged

(PE-38685) Beaker 6 upgrade #281

merged 2 commits into from
Jun 21, 2024

Conversation

tlehman
Copy link
Contributor

@tlehman tlehman commented Jun 21, 2024

What's this PR do?

Upgrades beaker to 6, kicks off a release (3.2 -> 3.3)

Any background context you want to provide?

As part of the beaker 6 upgrade for pe_acceptance_tests, I had to bump the version on beaker-pe. My local testing of pe_acceptance_tests pointed at my local directory for beaker-pe, so I verified this works.

Details of my test:

In pe_acceptance_tests, locally, I pointed beaker-pe at my local directory (mentioned above), then pointed at beaker-puppet 4.0 (to get the beaker 6 transitive deps worked out).

Then I visited this CI run and yoinked a set of commands to run locally:

  1. beaker-hostgenerator
  2. beaker init
  3. pe_ver and pe_dir for the hosts.cfg

The bundle exec beaker exec command completed with a return value of 0, all tests passing. :shipit:

As part of the beaker 6 upgrade for pe_acceptance_tests, I had to bump the version on beaker-pe.
My local testing of pe_acceptance_tests pointed at my local directory for beaker-pe, so I verified this
works.
@tlehman tlehman requested review from a team as code owners June 21, 2024 20:27
@donoghuc
Copy link
Member

Test failures are unrealted (pulling agents from wrong location). Going to merge and release to unblock beaker 6 upgrade in integration test.

@donoghuc donoghuc merged commit dd9e1fc into puppetlabs:main Jun 21, 2024
2 of 4 checks passed
@tlehman tlehman deleted the PE38685 branch June 21, 2024 23:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants