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

openshift-installer-presubmits: Separate sequence entry for --action_env #1190

Conversation

wking
Copy link
Member

@wking wking commented Aug 14, 2018

Avoid:

sh: 0: Can't open ./hack/test-bazel-build-tarball.sh --action_env=HOME=/tmp

This PR fixes a typo from #1185.

/assign @smarterclayton.

@openshift-ci-robot openshift-ci-robot added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Aug 14, 2018
@smarterclayton
Copy link
Contributor

/lgtm

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Aug 14, 2018
@openshift-merge-robot openshift-merge-robot merged commit c5a7857 into openshift:master Aug 14, 2018
@openshift-ci-robot
Copy link
Contributor

@wking: Updated the job-config configmap using the following files:

  • key openshift-installer-presubmits.yaml using file ci-operator/jobs/openshift/installer/openshift-installer-presubmits.yaml

In response to this:

Avoid:

sh: 0: Can't open ./hack/test-bazel-build-tarball.sh --action_env=HOME=/tmp

This PR fixes a typo from #1185.

/assign @smarterclayton.

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.

@wking wking deleted the installer-bazel-tarball-action-env-split branch August 14, 2018 21:40
derekhiggins pushed a commit to derekhiggins/release that referenced this pull request Oct 24, 2023
In recent (2.2.1 tested) podman versions, this file ends up
unreadable as $USER e.g in utils.sh

$ sudo podman login --authfile /tmp/test.json ...
Login Succeeded!
-rw-------. 1 root root 107 Feb 17 12:00 /tmp/test.json

In future we should probably test removing all the `sudo podman`
calls to use rootless mode, but since that has historically been
buggy, this is a lower risk interim fix.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lgtm Indicates that a PR is ready to be merged. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants