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

Update pip in GHA, rollback PR #9341 #9357

Merged
merged 2 commits into from Feb 8, 2022

Conversation

Gauravtalreja1
Copy link
Contributor

@Gauravtalreja1 Gauravtalreja1 commented Feb 8, 2022

Description:

  1. Upstream pypa/pip issue[Fix assertion error when determining installation order. pypa/pip#10867] for pip22 is solved, for which we temporarily disabled updating pip in GHA
  2. Tests collection for tests/upgrades fails in GHA for recently added SATUTILS_REPO and SATCLIENT_REPO variables, check Added client and utils repository support in satellite capsule and client upgrade. satellite6-upgrade#521

Tested here: Gauravtalreja1#2

Signed-off-by: Gaurav Talreja <gtalreja@redhat.com>
Signed-off-by: Gaurav Talreja <gtalreja@redhat.com>
@@ -33,6 +33,14 @@ REPOS:
RHEL7: replace-with-rhel7-http-link
RHEL8: replace-with-rhel8-http-link
RHEL9: replace-with-rhel9-http-link
# Satellite's utils repos, and it works for 7.0 and onwards
Copy link
Member

Choose a reason for hiding this comment

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

This is a really ugly crossover in validation of the yamls, since these fields were introduced by sat6-upgrade. Not directly related to this PR or needing to be fixed in this PR. But the fact that GHA will fail for robottelo PRs because satellite6-upgrade added some fields to its repos.yaml is broken.

Copy link
Member

Choose a reason for hiding this comment

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

Yeah! Agree but the checks were failing for upgrade scenarios collections where it's looking for setting in robottelo's confs.

@mshriver mshriver merged commit b6d7499 into SatelliteQE:master Feb 8, 2022
@Gauravtalreja1 Gauravtalreja1 deleted the fix-gha branch October 19, 2022 09:18
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.

None yet

3 participants