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
Bug 1904065: [on-prem] export proxy variables to be taken in account #2279
Bug 1904065: [on-prem] export proxy variables to be taken in account #2279
Conversation
@EmilienM: This pull request references Bugzilla bug 1873556, which is invalid:
Comment 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. |
@EmilienM: This pull request references Bugzilla bug 1873556, which is invalid:
Comment 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. |
/assign cgwalters |
I'll refresh BZ bot once it pass QA |
@EmilienM: This pull request references Bugzilla bug 1904065, which is invalid:
Comment 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. |
3155460
to
eb6533c
Compare
/bugzilla refresh |
@EmilienM: This pull request references Bugzilla bug 1904065, which is invalid:
Comment 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. |
/bugzilla refresh |
@mandre: This pull request references Bugzilla bug 1904065, which is valid. The bug has been moved to the POST state. The bug has been updated to refer to the pull request using the external bug tracker. 6 validation(s) were run on this bug
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. |
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.
We'll need to fix all on-prem platforms, not just baremetal.
- https://github.com/openshift/machine-config-operator/blob/release-4.6/templates/common/openstack/files/NetworkManager-resolv-prepender.yaml
- https://github.com/openshift/machine-config-operator/blob/release-4.6/templates/common/ovirt/files/NetworkManager-resolv-prepender.yaml
- https://github.com/openshift/machine-config-operator/blob/release-4.6/templates/common/vsphere/files/NetworkManager-resolv-prepender.yaml
/retest |
/bugzilla refresh |
@kikisdeliveryservice: This pull request references Bugzilla bug 1904065, which is valid. 6 validation(s) were run on this bug
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. |
/retest |
In order to have the proxy variables (HTTP_PROXY, HTTPS_PROXY and NO_PROXY), we need to `export` them otherwise then don't end up being loaded in the environment and it causes issues if a proxy is used, when pulling an image from a registry for example. (cherry picked from commit f41b1d2) Signed-off-by: Emilien Macchi <emilien@redhat.com>
eb6533c
to
5e657a4
Compare
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
@kikisdeliveryservice hi, the PR is now updated, and IMO ready to merge. Thanks |
/lgtm |
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: bcrochet, EmilienM, kikisdeliveryservice, mandre, pierreprinetti 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 |
/bugzilla refresh |
@eparis: This pull request references Bugzilla bug 1904065, which is valid. 6 validation(s) were run on this bug
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. |
/retest Please review the full test history for this PR and help us cut down flakes. |
1 similar comment
/retest Please review the full test history for this PR and help us cut down flakes. |
/skip |
@EmilienM: 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. |
/retest Please review the full test history for this PR and help us cut down flakes. |
this okd test is completely red but the bot doesn't listen to me.. /skip |
Hey, the bot listened that time! |
@EmilienM: All pull requests linked via external trackers have merged: Bugzilla bug 1904065 has been moved to the MODIFIED state. 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. |
/cherry-pick release-4.5 |
@cybertron: new pull request created: #2301 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. |
Note: manual cherry-pick of #2266 (bot can't handle git conflicts)
In order to have the proxy variables (HTTP_PROXY, HTTPS_PROXY and
NO_PROXY), we need to
export
them otherwise then don't end up beingloaded in the environment and it causes issues if a proxy is used,
when pulling an image from a registry for example.
Signed-off-by: Emilien Macchi emilien@redhat.com
(cherry picked from commit f41b1d2)