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

Remove support for Debian 9 and Ubuntu 16.04; Add Debian 10 support #928

Merged
merged 2 commits into from
May 11, 2023

Conversation

wyardley
Copy link
Contributor

@wyardley wyardley commented May 10, 2023

Pull Request (PR) description

This Pull Request (PR) fixes the following issues

Closes #844
Fixes #930
Fixes #887

@wyardley wyardley marked this pull request as ready for review May 10, 2023 19:41
@wyardley
Copy link
Contributor Author

@bastelfreak actually, this is not as bad as I thought. acceptance tests basically pass - I can see if I can figure out an explanation / fix for the one failing test.
Let me know if there's a better way to handle the title / description for both removing and adding support for versions in the same PR, or if it needs to be done in two phases.

  1) rabbitmq parameter on a vhost: create parameter resource runs successfully
     Failure/Error: apply_manifest(pp, catch_changes: true)
     Beaker::Host::CommandFailure:
       Host 'debian10-64-puppet7.example.com' exited with 2 running:
        puppet apply --verbose --detailed-exitcodes /tmp/apply_manifest_195729764.pp.4YjFap
       Last 10 lines of output were:
       	Info: Loading facts
       	Info: Loading facts
       	Notice: Compiled catalog for debian10-64-puppet7.example.com in environment production in 0.29 seconds
       	Info: Using environment 'production'
       	Info: Applying configuration version '1683748656'
       	Notice: /Stage[main]/Main/Rabbitmq_plugin[rabbitmq_federation]/ensure: created
       	Info: /Stage[main]/Main/Rabbitmq_plugin[rabbitmq_federation]: Scheduling refresh of Service[rabbitmq-server]
       	Notice: /Stage[main]/Rabbitmq::Service/Service[rabbitmq-server]: Triggered 'refresh' from 1 event
       	Info: Calling prefetch: documentumFed@fedhost
       	Notice: Applied catalog in 18.74 seconds

@bastelfreak
Copy link
Member

ah nice, thanks! I think the title is fine.

@wyardley
Copy link
Contributor Author

  Notice: Applied catalog in 61.59 seconds

debian10-64-1 executed in 89.26 seconds
Exited: 2
    runs successfully (FAILED - 3)

debian10-64-1 13:50:34$ rabbitmqctl list_parameters -p fedhost
  Listing runtime parameters for vhost "fedhost" ...
  federation-upstream	documentumFed	{"expires":3600000,"uri":"amqp://server"}

debian10-64-1 executed in 2.40 seconds
    has the parameter

So, if I run the manifest again locally, the exit code is 0.
Also, the actual test that the parameter exists is there.

Not sure if it's just an idempotency or timing issue? There's no output that suggests an actual problem.

@wyardley
Copy link
Contributor Author

Worse, it doesn't seem to be totally consistent what fails 🤦

@wyardley
Copy link
Contributor Author

Ubuntu 22 has a lot more failures; going to leave that out for now.
If anyone on IRC (or any folks who have been more involved recently with Vox / Puppet stuff) has time to try and fix modulesync, that would also be super helpful, because I am wondering if that might help make sure all the integration test scaffolding is up to date.

@wyardley wyardley added the enhancement New feature or request label May 11, 2023
@wyardley
Copy link
Contributor Author

Ok, pushed a test fix for some clustering related stuff.

The other stubborn one is the "create parameter resource" one, where the test itself works fine, but the idempotency check (no changes on second run) fails. Going to look at that one.

  Info: Applying configuration version '1683775408'
  Notice: /Stage[main]/Main/Rabbitmq_plugin[rabbitmq_federation]/ensure: created

The issue w/ Ubuntu 20.04 seems to be related to Python installation seemingly needing to run each time.

@wyardley
Copy link
Contributor Author

It looks like the main issue is due to the newer RabbitMQ version and the issue outlined in #844

@wyardley wyardley force-pushed the wyardley/deprecate branch 4 times, most recently from cf23711 to 6a84a2b Compare May 11, 2023 05:31
@wyardley wyardley force-pushed the wyardley/deprecate branch 2 times, most recently from f634292 to 908e80b Compare May 11, 2023 05:48
- Deprecate Debian 9 and Ubuntu 16.04
- Add support for Debian 10
- Remove RABBITMQ_USE_LONGNAME in clustering acceptance tests
  Acceptance tests fail on instance without a FQDN when
  `RABBITMQ_USE_LONGNAME` is set to `true` in the environment.
  Simplify the test case; this still seems to work on older and newer
  supported versions

Signed-off-by: William Yardley <[email protected]>
Rabbitmq_plugin now correctly detects implicitly enabled plugins to
preserve idempotency regardless of plugin install order

Cherry-picked from voxpupuli#844
Fixes voxpupuli#930

Signed-off-by: William Yardley <[email protected]>
@wyardley
Copy link
Contributor Author

wyardley commented May 11, 2023

I gave up on Debian 11 / Ubuntu 20 for today. Someone can sort those out after some of the rest of this gets out.

Arch failures were preexisting.

@bastelfreak bastelfreak changed the title BREAKING: Remove support for Debian 9 and Ubuntu 16.04 Remove support for Debian 9 and Ubuntu 16.04; Add Debian 10 support May 11, 2023
@bastelfreak bastelfreak merged commit 7bc0d66 into voxpupuli:master May 11, 2023
@wyardley wyardley mentioned this pull request May 13, 2023
@wyardley wyardley deleted the wyardley/deprecate branch May 19, 2024 17:48
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.

Idempotency issue with implicitly enabled plugins Add support for debian 10
3 participants