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.

Sign up for GitHub

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

Jump to bottom

Bump broker to 0.5 #15221

Merged
merged 3 commits into from
Jun 10, 2024
Merged

Bump broker to 0.5 #15221

merged 3 commits into from
Jun 10, 2024

Conversation

JacobCallahan
Copy link
Member

This requirement update also adds in the podman and hussh dependency groups.

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

@JacobCallahan
Bump broker to 0.5
de9b802 
This requirement update also adds in the podman and hussh dependency groups.
@JacobCallahan JacobCallahan added dependencies Pull requests that update a dependency file CherryPick PR needs CherryPick to previous branches AutoMerge_Cherry_Picked The cherrypicked PRs of master PR would be automerged if all checks passing 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 labels May 29, 2024
@JacobCallahan JacobCallahan requested a review from a team as a code owner May 29, 2024 18:18
@JacobCallahan JacobCallahan marked this pull request as draft May 29, 2024 18:19
@JacobCallahan
Update dependency_tests.yaml
3cfa9ee 
The broker dependency definition has new sub-dependencies with the 0.5.0 release.
@JacobCallahan
Copy link
Member Author

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

@JacobCallahan
Copy link
Member Author 

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

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 7184
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 : == 6 failed, 11 passed, 5478 deselected, 6734 warnings in 8007.45s (2:13:27) ===

@Satellite-QE Satellite-QE added the PRT-Failed Indicates that latest PRT run is failed for the PR label May 29, 2024
@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 7186
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 : == 6 failed, 11 passed, 5478 deselected, 6727 warnings in 7915.49s (2:11:55) ===

@Gauravtalreja1
Copy link
Collaborator 

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

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 7248
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 : == 6 failed, 11 passed, 5501 deselected, 6746 warnings in 7864.05s (2:11:04) ===

@JacobCallahan JacobCallahan marked this pull request as ready for review June 3, 2024 18:14
@Gauravtalreja1
Copy link
Collaborator

@JacobCallahan Did you get an chance to look at the PRT failures? Does this seem related?

@JacobCallahan
Copy link
Member Author

@Gauravtalreja1 those failures so not seem to be related. However, I would like @shubhamsg199 to run the tests he wanted Hussh for via PRT as well

@jyejare jyejare removed the 6.16.z label Jun 5, 2024
@Gauravtalreja1
Copy link
Collaborator 

trigger: test-robottelo
pytest: tests/foreman/api/test_provisioning.py -k test_rhel_pxe_provisioning
provisioning: true
env:
     BROKER_SSH_BACKEND: hussh

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 7321
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/api/test_provisioning.py -k test_rhel_pxe_provisioning --external-logging --include-onprem-provisioning
Test Result : === 4 failed, 8 passed, 13 deselected, 2627 warnings in 19093.37s (5:18:13) ====

Gauravtalreja1
Gauravtalreja1 approved these changes Jun 7, 2024
View reviewed changes
Copy link
Collaborator

@Gauravtalreja1 Gauravtalreja1 left a comment

Choose a reason for hiding this comment

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

LGTM, PRT failures for above provisioning tests are related to known bug for EL8 provisioning.

@Gauravtalreja1 Gauravtalreja1 mentioned this pull request Jun 8, 2024
Closed
shubhamsg199
shubhamsg199 approved these changes Jun 10, 2024
View reviewed changes
Copy link
Contributor

@shubhamsg199 shubhamsg199 left a comment

Choose a reason for hiding this comment

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

LGTM, Thanks @Gauravtalreja1 for testing provisioning. The failures are expected due to the RHEL8 provisioning bug.

@Gauravtalreja1 Gauravtalreja1 merged commit 9072e6f into master Jun 10, 2024
9 of 10 checks passed
@Gauravtalreja1 Gauravtalreja1 deleted the JacobCallahan-broker-05 branch June 10, 2024 10:08
github-actions bot pushed a commit that referenced this pull request Jun 10, 2024
@JacobCallahan @web-flow
Bump broker to 0.5 (#15221)
5cec282 
* Bump broker to 0.5

This requirement update also adds in the podman and hussh dependency groups.

* Update dependency_tests.yaml

The broker dependency definition has new sub-dependencies with the 0.5.0 release.

---------

Co-authored-by: Gaurav Talreja <[email protected]>
(cherry picked from commit 9072e6f)
@Satellite-QE Satellite-QE mentioned this pull request Jun 10, 2024
Merged
github-actions bot pushed a commit that referenced this pull request Jun 10, 2024
@JacobCallahan @web-flow
Bump broker to 0.5 (#15221)
370e7bf 
* Bump broker to 0.5

This requirement update also adds in the podman and hussh dependency groups.

* Update dependency_tests.yaml

The broker dependency definition has new sub-dependencies with the 0.5.0 release.

---------

Co-authored-by: Gaurav Talreja <[email protected]>
(cherry picked from commit 9072e6f)
This was referenced Jun 10, 2024
Merged
Merged
github-actions bot pushed a commit that referenced this pull request Jun 10, 2024
@JacobCallahan @web-flow
Bump broker to 0.5 (#15221)
52bfc0d 
* Bump broker to 0.5

This requirement update also adds in the podman and hussh dependency groups.

* Update dependency_tests.yaml

The broker dependency definition has new sub-dependencies with the 0.5.0 release.

---------

Co-authored-by: Gaurav Talreja <[email protected]>
(cherry picked from commit 9072e6f)
Gauravtalreja1 pushed a commit that referenced this pull request Jun 11, 2024
@JacobCallahan @Gauravtalreja1
Bump broker to 0.5 (#15221)
eb2a6f7 
* Bump broker to 0.5

This requirement update also adds in the podman and hussh dependency groups.

* Update dependency_tests.yaml

The broker dependency definition has new sub-dependencies with the 0.5.0 release.

---------

Co-authored-by: Gaurav Talreja <[email protected]>
(cherry picked from commit 9072e6f)
Gauravtalreja1 pushed a commit that referenced this pull request Jun 11, 2024
@Satellite-QE @JacobCallahan
[6.15.z] Bump broker to 0.5 (#15380)
dd64abe 
Bump broker to 0.5 (#15221)

* Bump broker to 0.5

This requirement update also adds in the podman and hussh dependency groups.

* Update dependency_tests.yaml

The broker dependency definition has new sub-dependencies with the 0.5.0 release.

---------

Co-authored-by: Gaurav Talreja <[email protected]>
(cherry picked from commit 9072e6f)

Co-authored-by: Jake Callahan <[email protected]>
Gauravtalreja1 pushed a commit that referenced this pull request Jun 11, 2024
@JacobCallahan @Gauravtalreja1
Bump broker to 0.5 (#15221)
60230f7 
* Bump broker to 0.5

This requirement update also adds in the podman and hussh dependency groups.

* Update dependency_tests.yaml

The broker dependency definition has new sub-dependencies with the 0.5.0 release.

---------

Co-authored-by: Gaurav Talreja <[email protected]>
(cherry picked from commit 9072e6f)
Gauravtalreja1 pushed a commit that referenced this pull request Jun 11, 2024
@JacobCallahan @Gauravtalreja1
Bump broker to 0.5 (#15221)
0853d91 
* Bump broker to 0.5

This requirement update also adds in the podman and hussh dependency groups.

* Update dependency_tests.yaml

The broker dependency definition has new sub-dependencies with the 0.5.0 release.

---------

Co-authored-by: Gaurav Talreja <[email protected]>
(cherry picked from commit 9072e6f)
Gauravtalreja1 pushed a commit that referenced this pull request Jun 11, 2024
@Satellite-QE @JacobCallahan
[6.14.z] Bump broker to 0.5 (#15379)
3f22b4a 
Bump broker to 0.5 (#15221)

* Bump broker to 0.5

This requirement update also adds in the podman and hussh dependency groups.

* Update dependency_tests.yaml

The broker dependency definition has new sub-dependencies with the 0.5.0 release.

---------

Co-authored-by: Gaurav Talreja <[email protected]>
(cherry picked from commit 9072e6f)

Co-authored-by: Jake Callahan <[email protected]>
Gauravtalreja1 pushed a commit that referenced this pull request Jun 11, 2024
@Satellite-QE @JacobCallahan
[6.13.z] Bump broker to 0.5 (#15381)
513a0b0 
Bump broker to 0.5 (#15221)

* Bump broker to 0.5

This requirement update also adds in the podman and hussh dependency groups.

* Update dependency_tests.yaml

The broker dependency definition has new sub-dependencies with the 0.5.0 release.

---------

Co-authored-by: Gaurav Talreja <[email protected]>
(cherry picked from commit 9072e6f)

Co-authored-by: Jake Callahan <[email protected]>
@dosas dosas mentioned this pull request Jun 18, 2024
Merged
jyejare pushed a commit to jyejare/robottelo that referenced this pull request Oct 19, 2024
@JacobCallahan @Gauravtalreja1 @jyejare
Bump broker to 0.5 (SatelliteQE#15221)
8ef54a9 
* Bump broker to 0.5

This requirement update also adds in the podman and hussh dependency groups.

* Update dependency_tests.yaml

The broker dependency definition has new sub-dependencies with the 0.5.0 release.

---------

Co-authored-by: Gaurav Talreja <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Gauravtalreja1 Gauravtalreja1 Gauravtalreja1 approved these changes

@shubhamsg199 shubhamsg199 shubhamsg199 approved these changes

Assignees

@JacobCallahan JacobCallahan

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 AutoMerge_Cherry_Picked The cherrypicked PRs of master PR would be automerged if all checks passing 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
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@JacobCallahan @Satellite-QE @Gauravtalreja1 @shubhamsg199 @jyejare