Closed philippthun closed 5 years ago
@ImJasonH - As this PR has been reviewed and approved by @evankanderson, would it be possible to merge it? Or is something missing from your point of view?
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: evankanderson, ImJasonH, philippthun
The full list of commands accepted by this bot can be found here.
The pull request process is described here
/retest
@philippthun: Cannot trigger testing until a trusted user reviews the PR and leaves an /ok-to-test
message.
/ok-to-test
On Mon, Jan 28, 2019, 4:01 PM Knative Prow Robot notifications@github.com wrote:
@philippthun https://github.com/philippthun: Cannot trigger testing until a trusted user reviews the PR and leaves an /ok-to-test message.
In response to this https://github.com/knative/build-templates/pull/81#issuecomment-458351869 :
/retest
Instructions for interacting with me using PR comments are available here https://git.k8s.io/community/contributors/guide/pull-requests.md. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra https://github.com/kubernetes/test-infra/issues/new?title=Prow%20issue: repository.
— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/knative/build-templates/pull/81#issuecomment-458351994, or mute the thread https://github.com/notifications/unsubscribe-auth/AHlyN8YDduMOD_2k452kgABc9m9m-VXeks5vH49TgaJpZM4ZGgff .
When executing the
ecr_helper.sh
(on Ubuntu 18.04 with GNU bash version 4.4.19(1)-release) we get an error due to quoting issues caused by manipulating the Internal Field Separator:To fix this issue, we simply
unset IFS
as soon as username and password have been parsed from the token.