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

Downgrade testing-farm to 0.0.16 #633

Merged
merged 1 commit into from
Aug 7, 2024
Merged

Downgrade testing-farm to 0.0.16 #633

merged 1 commit into from
Aug 7, 2024

Conversation

kwk
Copy link
Collaborator

@kwk kwk commented Aug 7, 2024

This should allow us to get artifact URLs pointing to testing-farm results on the redhat ranch.

In 0.0.18 or before, testing-farm introduced more detailed output in the
testing-farm watch command which required to make calls that at least
the redhat ranch were only possible in the Red Hat VPN which is not
possible on github.

Here's the change: https://gitlab.com/testing-farm/cli/-/compare/v0.0.18...v0.0.17?from_project_id=19011083&straight=true#e8477e57acb5b45efd72f21c9596bee8b05a8c15_283_200

This should allow us to get artifact URLs pointing to testing-farm results on the redhat ranch.

In 0.0.18 or before, testing-farm introduced more detailed output in the
`testing-farm watch` command which required to make calls that at least
the redhat ranch were only possible in the Red Hat VPN which is not
possible on github.
@kwk kwk requested a review from nikic August 7, 2024 13:49
Copy link
Collaborator

@nikic nikic left a comment

Choose a reason for hiding this comment

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

LGTM

@kwk kwk merged commit 5bd544b into main Aug 7, 2024
4 of 6 checks passed
@kwk kwk deleted the downgrade-testing-farm branch August 7, 2024 13:50
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.

2 participants