8000 cmd, pkg/utils: Split out the code to get the initialization stamp path by debarshiray · Pull Request #1633 · containers/toolbox · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

cmd, pkg/utils: Split out the code to get the initialization stamp path #1633

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

Conversation

debarshiray
Copy link
Member

This will prevent any silly bug in getting the initialization stamp path
from breaking the communication protocol between the enter or run
commands on the host and the Toolbx container's entry point process.

This is meant to reduce the size of the initContainer() function that
implements the heart of the 'init-container' command.

containers#1631
This is meant to reduce the size of the initContainer() function that
implements the heart of the 'init-container' command.

The debug log and error message were tweaked to match the name of the
function and for consistency with the configureRPM() function.

containers#1631
This will prevent any silly bug in getting the initialization stamp path
from breaking the communication protocol between the 'enter' or 'run'
commands on the host and the Toolbx container's entry point process.

containers#1633
@debarshiray debarshiray force-pushed the wip/rishi/cmd-pkg-utils-split-out-initialization-stamp-path branch from 0c154a3 to 2956eca Compare April 30, 2025 22:14
Copy link

@debarshiray debarshiray merged commit 2956eca into containers:main May 1, 2025
3 checks passed
@debarshiray debarshiray deleted the wip/rishi/cmd-pkg-utils-split-out-initialization-stamp-path branch May 1, 2025 01:31
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.

1 participant
0