-
Notifications
You must be signed in to change notification settings - Fork 189
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
USHIFT-1411: pin ovs dependency to 3.1.1 and fix up CI #2016
USHIFT-1411: pin ovs dependency to 3.1.1 and fix up CI #2016
Conversation
@dhellmann: This pull request references USHIFT-1411 which is a valid jira issue. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
7693273
to
b0664d1
Compare
/test microshift-metal-tests |
/hold It is not clear this is actually helpful. |
b0664d1
to
a44cddc
Compare
/hold cancel |
/test microshift-metal-tests |
This is a temporary work-around for OCPBUGS-15948
a44cddc
to
1441e34
Compare
/test microshift-metal-tests |
/test microshift-metal-tests |
2 similar comments
/test microshift-metal-tests |
/test microshift-metal-tests |
/test verify |
/test microshift-metal-tests |
Makefile
Outdated
@@ -136,7 +136,7 @@ verify-fast: verify-go verify-assets verify-sh verify-py verify-config verify-rf | |||
|
|||
# Full verification checks that should run in CI | |||
.PHONY: verify-ci | |||
verify-ci: verify-fast verify-images verify-licenses verify-containers | |||
verify-ci: verify-fast verify-images verify-containers |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why was the license verification removed?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The commit message for that commit shows the error message that is breaking the job.
2023/07/08 18:18:45 failed to evaluate licenses: invalid character 'G' looking for beginning of value
It affects all PRs, so I suspect an updated version of the tool is behaving differently. We don't add new dependencies often, so I thought rather than spending time figuring out why that tool is reporting an error I would just turn it off.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
See #1987 (comment) for example
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Interesting. This problem does not occur on my development machine.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I was not able to reproduce it locally either. Perhaps it has something to do with the compiler version? Either way, I wanted to unblock us without spending time on a low-value tool.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I implemented a w/a in the #2020 PR
/test microshift-metal-tests |
/test microshift-metal-tests
|
/test microshift-metal-tests |
Maybe we need something like |
bf6e5a1
to
eac0e4e
Compare
Try to avoid dnf conflicts by clearing the cache before we start doing anything real. Also show dnf history to debug cache collisions from parallel operations.
eac0e4e
to
74c6855
Compare
/test microshift-metal-tests |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
/hold
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: dhellmann, pacevedom The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
/hold cancel |
@dhellmann: The following tests failed, say
Full PR test history. Your PR dashboard. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here. |
This is a temporary work-around for OCPBUGS-15948
/assign @pmtk @pliurh
/cc @zshi-redhat @pacevedom