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

Bump broker[docker,hussh,podman] from 0.5.7 to 0.6.2 #16691

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

dependabot[bot]
Copy link
Contributor

@dependabot dependabot bot commented on behalf of github Oct 17, 2024

Bumps broker[docker,hussh,podman] from 0.5.7 to 0.6.2.

Release notes

Sourced from broker[docker,hussh,podman]'s releases.

The release that improved the new user experience

What's Changed

Full Changelog: SatelliteQE/broker@0.6.1...0.6.2

The release that caught some usability issues

What's Changed

Thanks to @​bherrin3 for identifying the json/yaml formatting issue

Full Changelog: SatelliteQE/broker@0.6.0...0.6.1

The release that made config great

What's Changed

This is a larger release mainly focused on the configuration user experience as well as the CLI presentation layer. Please check the wiki for 0.6-specific features and upgrade steps.

Small workaround for container syncing #325 by JacobCallahan

Rework inventory cli #324 by JacobCallahan

Enable timeouts for command execution when using Hussh #322 by JacobCallahan

Full Changelog: SatelliteQE/broker@0.5.7...0.6.0

Commits
  • 03456dc Add checks for bad input for AT provider help paths
  • 27a9edb Remove deprecated method for pulling AnsibleTower host facts
  • ecd66d6 More fixes
  • 55d312a Bump pypa/gh-action-pypi-publish from 1.10.1 to 1.10.3
  • 37ae95b Quick fix for https config sources
  • b8f46f1 Small workaround for container syncing
  • c37ea1c Update broker/helpers.py
  • 54b13f7 Adjust a couple of migrations
  • f26a40d Add in the sad ability to lessen color output for rich
  • c1d0c7d Rework inventory cli
  • Additional commits viewable in compare view

Dependabot compatibility score

You can trigger a rebase of this PR by commenting @dependabot rebase.


Dependabot commands and options

You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot merge will merge this PR after your CI passes on it
  • @dependabot squash and merge will squash and merge this PR after your CI passes on it
  • @dependabot cancel merge will cancel a previously requested merge and block automerging
  • @dependabot reopen will reopen this PR if it is closed
  • @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
  • @dependabot show <dependency name> ignore conditions will show all of the ignore conditions of the specified dependency
  • @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)

Note
Automatic rebases have been disabled on this pull request as it has been open for over 30 days.

@dependabot dependabot bot requested a review from a team as a code owner October 17, 2024 03:26
@dependabot dependabot bot added 6.13.z Introduced in or relating directly to Satellite 6.13 6.14.z Introduced in or relating directly to Satellite 6.14 6.15.z Introduced in or relating directly to Satellite 6.15 6.16.z Introduced in or relating directly to Satellite 6.16 CherryPick PR needs CherryPick to previous branches dependencies Pull requests that update a dependency file labels Oct 17, 2024
Copy link

trigger: test-robottelo\r
pytest: tests/foreman/ -k 'test_host_registration_end_to_end or test_positive_erratum_applicability or test_positive_upload_content'

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 8985
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/ -k test_host_registration_end_to_end or test_positive_erratum_applicability or test_positive_upload_content --external-logging
Test Result : ======= 5449 deselected, 5471 warnings, 18 errors in 1931.49s (0:32:11) ========

@Satellite-QE Satellite-QE added the PRT-Failed Indicates that latest PRT run is failed for the PR label Oct 17, 2024
@dependabot dependabot bot force-pushed the dependabot/pip/broker-dockerhusshpodman--0.6.2 branch from 09bfe87 to f1022d4 Compare October 21, 2024 04:04
Copy link

trigger: test-robottelo\r
pytest: tests/foreman/ -k 'test_host_registration_end_to_end or test_positive_erratum_applicability or test_positive_upload_content'

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 9033
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/ -k test_host_registration_end_to_end or test_positive_erratum_applicability or test_positive_upload_content --external-logging
Test Result : ======== 5451 deselected, 5473 warnings, 18 errors in 556.52s (0:09:16) ========

@JacobCallahan
Copy link
Member

@omkarkhatavkar @jyejare can you determine why the PRT runs aren't actually installing the updated version of the requirements.txt? Is that an explicit step we need to add to the PRT job setup?

Mandatory Requirements Mismatch: ... broker[docker,podman,hussh]==0.6.2 ...

@jyejare
Copy link
Member

jyejare commented Oct 22, 2024

@JacobCallahan It is installing the updated requirements but the pytest plugin for mismatch reqs was not updated for the recent changes.

The PR is raised in robottelo to handle that and you won't see those warnings again.

@JacobCallahan
Copy link
Member

@jyejare The pip freeze output in artifacts still shows broker==0.5.7. Additionally, log lines don't match up with their correct position in Broker 0.6.2. Can we verify that we are actually re-running an install of requirements.txt?

@jyejare
Copy link
Member

jyejare commented Oct 23, 2024

@JacobCallahan The build artifacts / logs showing 0.6.2. Are they showing for you locally?

log lines don't match up with their correct position in Broker 0.6.2

Do you mean the name of the broker as a requirement broker[docker,podman,hussh] is different from what in pip feeeze Broker? If yes that is handled in #16692

Can we verify that we are actually re-running an install of requirements.txt?

In CI checks we are not updating the requirements, we are installing all fresh and its working as expected I see. But with req update plugin, we need to mention the option to update all / required reqs then only it updates. Also the update only happens for mismatch requirements and git repositories. All things are now improved in #16692

I may misunderstood something here and open to here from you back.

@Gauravtalreja1
Copy link
Collaborator

@JacobCallahan Do we still have requirements mismatch issues with this PR? Just curious what's pending on this update?

@JacobCallahan
Copy link
Member

trigger: test-robottelo
pytest: tests/foreman/ -k 'test_host_registration_end_to_end or test_positive_erratum_applicability or test_positive_upload_content'

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 9375
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/ -k test_host_registration_end_to_end or test_positive_erratum_applicability or test_positive_upload_content --external-logging
Test Result : ======= 5462 deselected, 5484 warnings, 18 errors in 1655.43s (0:27:35) ========

@JacobCallahan
Copy link
Member

JacobCallahan commented Nov 18, 2024

All of the failures here look to be related to the disconnect between the current config and new config.
I've updated the CI MR to account for these other config changes, but it won't help this PR until it has been merged.

@JacobCallahan JacobCallahan self-assigned this Nov 19, 2024
@jyejare
Copy link
Member

jyejare commented Nov 25, 2024

@JacobCallahan We have ACKed the CI MR, but the lining is failing in the CI MR blocking it from merging and proceeding further on that MRs / this PRs merge.

@devendra104
Copy link
Member

@JacobCallahan Can we rebase the branch? The Jenkins MR's component pipelines are failing due to the missing ipa.group_users config?

15:49:58      raise ValidationError(_message, details=[(self, _message)])
15:49:58  dynaconf.validator.ValidationError: ipa.group_users is required in env main

@devendra104
Copy link
Member

@JacobCallahan Can we rebase the branch? The Jenkins MR's component pipelines are failing due to the missing ipa.group_users config?

15:49:58      raise ValidationError(_message, details=[(self, _message)])
15:49:58  dynaconf.validator.ValidationError: ipa.group_users is required in env main

These changes #16536 must be in this pr.

@jyejare
Copy link
Member

jyejare commented Nov 28, 2024

@dependabot rebase

Bumps [broker[docker,hussh,podman]](https://github.com/SatelliteQE/broker) from 0.5.7 to 0.6.2.
- [Release notes](https://github.com/SatelliteQE/broker/releases)
- [Commits](SatelliteQE/broker@0.5.7...0.6.2)

---
updated-dependencies:
- dependency-name: broker[docker,hussh,podman]
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>
@dependabot dependabot bot force-pushed the dependabot/pip/broker-dockerhusshpodman--0.6.2 branch from f1022d4 to 1a65b8d Compare November 28, 2024 07:16
@SatelliteQE SatelliteQE deleted a comment from github-actions bot Nov 28, 2024
@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 9475
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/ -k test_host_registration_end_to_end or test_positive_erratum_applicability or test_positive_upload_content --external-logging
Test Result : ======== 5521 deselected, 5545 warnings, 20 errors in 493.60s (0:08:13) ========

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
6.13.z Introduced in or relating directly to Satellite 6.13 6.14.z Introduced in or relating directly to Satellite 6.14 6.15.z Introduced in or relating directly to Satellite 6.15 6.16.z Introduced in or relating directly to Satellite 6.16 CherryPick PR needs CherryPick to previous branches dependencies Pull requests that update a dependency file PRT-Failed Indicates that latest PRT run is failed for the PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants