Skip to content

Comments

dracut/99emergency-timeout/timeout.sh: fix heredoc leak into arguments#33

Merged
pothos merged 2 commits intoflatcar-masterfrom
kai/cat-no-heredoc-as-argument
Jan 12, 2022
Merged

dracut/99emergency-timeout/timeout.sh: fix heredoc leak into arguments#33
pothos merged 2 commits intoflatcar-masterfrom
kai/cat-no-heredoc-as-argument

Conversation

@pothos
Copy link
Member

@pothos pothos commented Jan 12, 2022

  • dracut/99emergency-timeout/timeout.sh: fix heredoc leak into arguments

    The content that was supposed to be in the heredoc got interpreted as
    parameters for "cat", causing it to fail.
    Move the separator dahes into the next line to make sure it gets
    treated as heredoc content.

  • dracut/99emergency-timeout/timeout.sh: replace old CoreOS link

    The old CoreOS online validator does not have a new Flatcar home yet.
    Use the repo link for the binary of it instead.

How to use

Create coreos-overlay PR to update the commit reference and add the changelog there

Testing done

None, just manual execution of the moved line

  • Changelog entries added in the respective changelog/ directory (user-facing change, bug fix, security fix, update)
    ↑ Not done here, should be done in the coreos-overlay PR that pulls it in

The content that was supposed to be in the heredoc got interpreted as
parameters for "cat", causing it to fail.
Move the separator dahes into the next line to make sure it gets
treated as heredoc content.
The old CoreOS online validator does not have a new Flatcar home yet.
Use the repo link for the binary of it instead.
@pothos pothos requested a review from a team January 12, 2022 14:42
pothos added a commit to flatcar-archive/coreos-overlay that referenced this pull request Jan 12, 2022
@pothos pothos merged commit 18384ea into flatcar-master Jan 12, 2022
@pothos pothos deleted the kai/cat-no-heredoc-as-argument branch January 12, 2022 15:07
pothos added a commit to flatcar-archive/coreos-overlay that referenced this pull request Jan 12, 2022
pothos added a commit to flatcar-archive/coreos-overlay that referenced this pull request Jan 12, 2022
@dongsupark dongsupark added the bug Something isn't working label Jan 14, 2022
pothos added a commit to flatcar-archive/coreos-overlay that referenced this pull request Apr 26, 2022
This pulls in
flatcar/bootengine#33
to fix the error message printing when Ignition fails.
t-lo pushed a commit to flatcar/scripts that referenced this pull request Apr 13, 2023
This pulls in
flatcar/bootengine#33
to fix the error message printing when Ignition fails.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants