Skip to content

Conversation

@anmaxvl
Copy link
Contributor

@anmaxvl anmaxvl commented Mar 20, 2022

GCS init fails to parse the entropy socket when enabling ConsolePipe.
Fix by separating entropy parameters from the rest init args and
wrapping only init args with sh -c.

Signed-off-by: Maksim An [email protected]

GCS init fails to parse the entropy socket when enabling ConsolePipe.
Fix by separating entropy parameters from the rest init args and
wrapping only init args with `sh -c`.

Signed-off-by: Maksim An <[email protected]>
@anmaxvl anmaxvl requested a review from a team as a code owner March 20, 2022 06:13
@dcantah
Copy link
Contributor

dcantah commented Mar 21, 2022

Looks like this changes from

/init sh -c "-e 1 /bin/vsockexec -e 109 /bin/gcs -v4 -log-format json -loglevel debug & exec sh"

to

/init -e 1 sh -c "/bin/vsockexec -e 109 /bin/gcs -v4 -log-format json -loglevel debug & exec sh"

is that right?

@dcantah
Copy link
Contributor

dcantah commented Mar 21, 2022

Why does github keep posting everything I comment twice...

@dcantah dcantah self-assigned this Mar 21, 2022
@anmaxvl
Copy link
Contributor Author

anmaxvl commented Mar 21, 2022

Looks like this changes from

/init sh -c "-e 1 /bin/vsockexec -e 109 /bin/gcs -v4 -log-format json -loglevel debug & exec sh"

to

/init -e 1 sh -c "/bin/vsockexec -e 109 /bin/gcs -v4 -log-format json -loglevel debug & exec sh"

is that right?

yup.

@anmaxvl anmaxvl merged commit dfe9b5e into microsoft:master Mar 22, 2022
princepereira pushed a commit to princepereira/hcsshim that referenced this pull request Aug 29, 2024
GCS init fails to parse the entropy socket when enabling ConsolePipe.
Fix by separating entropy parameters from the rest init args and
wrapping only init args with `sh -c`.

Changes:
`/init sh -c "-e 1 /bin/vsockexec -e 109 /bin/gcs -v4 -log-format json -loglevel debug & exec sh"`
to
`/init -e 1 sh -c "/bin/vsockexec -e 109 /bin/gcs -v4 -log-format json -loglevel debug & exec sh"`

Signed-off-by: Maksim An <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants