-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
Revert --env-file changes #20256
Revert --env-file changes #20256
Conversation
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: Luap99 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 |
This reverts commit 7ce654f. see containers#19565 Signed-off-by: Paul Holzinger <[email protected]>
This reverts commit c67ef7c. see containers#19565 Signed-off-by: Paul Holzinger <[email protected]>
This reverts commit 170a786. This was a breaking change and users are hitting it, see containers#19565 Fixes containers#19565 Signed-off-by: Paul Holzinger <[email protected]>
Now that the newline env file change is reverted we have to adapt the tests. Signed-off-by: Paul Holzinger <[email protected]>
@edsantiago PTAL at test changes. |
Test and changes LGTM. Thank you for taking care of this. |
LGTM |
/lgtm |
This caused regressions for users, see #19565
Fixes #19565
Does this PR introduce a user-facing change?