-
Notifications
You must be signed in to change notification settings - Fork 126
CI: Re-enable buildextend testing for COSA CI #890
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
Conversation
|
/hold This contains a change that we do not want but need to be able to test this here. |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: travier The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
OK. I think we really need to have something like a Then |
45ba13b to
e1843be
Compare
|
Good idea. For now we can get away with that as this is mostly the only place where we don't build those in parallel and from the jobspec. |
|
/retest |
|
OK so, something just plain broke in CI here in fallout from some of the recent Prow rework it looks like. |
|
/retest |
|
#891 will fix our immediate issue and then we can figure out the failures in this one without having other PRs blocked on it. |
3c1bb88 to
bd74ac2
Compare
|
This one took more that 3 hours to run, so we'll need to remove some tests or have them run in parallel |
bd74ac2 to
cc6d6b1
Compare
Ensure we have a qemu image for followup builds. Fixes: c6e7dd9 Fixes: coreos/coreos-assembler#2973
cc6d6b1 to
314e804
Compare
|
@travier: The following test failed, say
Full PR test history. Your PR dashboard. DetailsInstructions 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 one is still taking too long. We will probably need to setup another test job that focuses on buildextends tests. |
|
Issues go stale after 90d of inactivity. Mark the issue as fresh by commenting If this issue is safe to close now please do so with /lifecycle stale |
|
Stale issues rot after 30d of inactivity. Mark the issue as fresh by commenting If this issue is safe to close now please do so with /lifecycle rotten |
|
Rotten issues close after 30d of inactivity. Reopen the issue by commenting /close |
|
@openshift-bot: Closed this PR. DetailsIn 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. |
This should fix coreos/coreos-assembler#2919 (comment)
Fixes: c6e7dd9
Fixes: coreos/coreos-assembler#2973