Skip to content

ci: Also work around git checkout security check#810

Merged
cgwalters merged 1 commit intoopenshift:masterfrom
cgwalters:ci-git
Jun 1, 2022
Merged

ci: Also work around git checkout security check#810
cgwalters merged 1 commit intoopenshift:masterfrom
cgwalters:ci-git

Conversation

@cgwalters
Copy link
Copy Markdown
Member

See actions/checkout#760 (comment)

Nothing malicious is happening here, we are just using the checkout
with various UIDs for complex reasons.

The name of this script is now kind of wrong, but changing it
would require a ratchet with the CI config in openshift/release.

See actions/checkout#760 (comment)

Nothing malicious is happening here, we are just using the checkout
with various UIDs for complex reasons.

The name of this script is now kind of wrong, but changing it
would require a ratchet with the CI config in openshift/release.
@cgwalters
Copy link
Copy Markdown
Member Author

Should fix the failures seen in #803 - but I didn't test that.

@openshift-ci openshift-ci Bot requested review from jmarrero and travier May 31, 2022 21:15
@openshift-ci openshift-ci Bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label May 31, 2022
Copy link
Copy Markdown
Member

@jmarrero jmarrero left a comment

Choose a reason for hiding this comment

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

lgtm

@travier
Copy link
Copy Markdown
Member

travier commented Jun 1, 2022

+ cosa buildextend-extensions
Running: rpm-ostree compose extensions --repo=/tmp/cosa/tmp/repo --touch-if-changed tmp/extensions/changed --cachedir=/tmp/cosa/cache src/config/manifest.yaml src/config/extensions.yaml --base-rev 957df6093862eb23194d7230948e43ed5142335e347eb3080e312027cf5b9918 --output-dir tmp/extensions/output
info: Missing CAP_SYS_ADMIN; using virt
Checking out 957df60... done!
warning: Found bdb_ro Packages database while attempting sqlite backend: using bdb_ro backend.
�[0m�[31merror: �[0mUnknown rpm-md repository: rhel-86-appstream

@cgwalters
Copy link
Copy Markdown
Member Author

Unknown rpm-md repository: rhel-86-appstream

Should be fixed by #802 hopefully.

/test test-qemu-kola-upgrade

@cgwalters
Copy link
Copy Markdown
Member Author

ci/prow/test-qemu-kola-upgrade — Job succeeded.           

🎉

@cgwalters
Copy link
Copy Markdown
Member Author

Since we know this fixes one job
/override ci/prow/test-qemu-metal
/override ci/prow/test-qemu-nvme
/override ci/prow/validate-built-image

@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci Bot commented Jun 1, 2022

@cgwalters: Overrode contexts on behalf of cgwalters: ci/prow/test-qemu-metal, ci/prow/test-qemu-nvme, ci/prow/validate-built-image

Details

In response to this:

Since we know this fixes one job
/override ci/prow/test-qemu-metal
/override ci/prow/test-qemu-nvme
/override ci/prow/validate-built-image

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.

@miabbott
Copy link
Copy Markdown
Member

miabbott commented Jun 1, 2022

/lgtm

@openshift-ci openshift-ci Bot added the lgtm Indicates that a PR is ready to be merged. label Jun 1, 2022
@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci Bot commented Jun 1, 2022

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: cgwalters, HuijingHei, jmarrero, miabbott

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:
  • OWNERS [HuijingHei,cgwalters,jmarrero,miabbott]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-ci-robot
Copy link
Copy Markdown

/retest-required

Remaining retests: 2 against base HEAD 16f7668 and 8 for PR HEAD fe25079 in total

@cgwalters
Copy link
Copy Markdown
Member Author

/skip

@cgwalters
Copy link
Copy Markdown
Member Author

/override ci/prow/periodic-images

@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci Bot commented Jun 1, 2022

@cgwalters: Overrode contexts on behalf of cgwalters: ci/prow/periodic-images

Details

In response to this:

/override ci/prow/periodic-images

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.

@cgwalters cgwalters merged commit c6bf5a9 into openshift:master Jun 1, 2022
@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci Bot commented Jun 1, 2022

@cgwalters: all tests passed!

Full PR test history. Your PR dashboard.

Details

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.

@cgwalters
Copy link
Copy Markdown
Member Author

/cherrypick release-4.11

@openshift-cherrypick-robot
Copy link
Copy Markdown

@cgwalters: new pull request created: #814

Details

In response to this:

/cherrypick release-4.11

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. lgtm Indicates that a PR is ready to be merged.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants