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

Updated locator to fix 2.8 clusterGroup failures #242

Merged
merged 2 commits into from
Dec 16, 2024
Merged

Conversation

sbulage
Copy link
Collaborator

@sbulage sbulage commented Dec 5, 2024

Fixes: #240

@sbulage sbulage added the automation Add or update automation label Dec 5, 2024
@sbulage sbulage self-assigned this Dec 5, 2024
@sbulage
Copy link
Collaborator Author

sbulage commented Dec 5, 2024

Before creating this PR, I ran CI 2.8 run --> https://github.com/rancher/fleet-e2e/actions/runs/12180011742

Here is the result:
Screenshot from 2024-12-05 21-24-18

Wait for the 2.9 and 2.10 result.

@mmartin24
Copy link
Collaborator

@sbulage something did not go well in the 2.10 and 2.9 ci's. I will hold the approval for the time being, ok?

@sbulage
Copy link
Collaborator Author

sbulage commented Dec 6, 2024

Above PR CI 2.9 result:

image

@sbulage
Copy link
Collaborator Author

sbulage commented Dec 6, 2024

@sbulage
Copy link
Collaborator Author

sbulage commented Dec 6, 2024

@sbulage
Copy link
Collaborator Author

sbulage commented Dec 6, 2024

@sbulage something did not go well in the 2.10 and 2.9 ci's. I will hold the approval for the time being, ok?

Fixed the failing part for all versions.

@mmartin24 mmartin24 self-requested a review December 12, 2024 06:45
Copy link
Collaborator

@mmartin24 mmartin24 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 for the adjustments.

@sbulage sbulage merged commit 01f771c into main Dec 16, 2024
17 of 18 checks passed
@sbulage sbulage deleted the fix_2.8_locator branch December 16, 2024 08:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automation Add or update automation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

CI-2.8: ClusterGroup tests are failing due to locator issue.
2 participants