Skip to content
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

fix: use entrypoint/command from template when possibile #29

Merged

Conversation

francesco-racciatti
Copy link
Collaborator

@francesco-racciatti francesco-racciatti commented Feb 12, 2024

What type of PR is this?

/kind bugfix

What this PR does / why we need it:

Use the entrypoint/command overridden by the template if available.

@francesco-racciatti francesco-racciatti force-pushed the fix-entrypoint-command-override-taskdefinition branch from e2c5166 to e2bfc9c Compare February 16, 2024 15:32
@francesco-racciatti francesco-racciatti marked this pull request as ready for review February 22, 2024 08:52
@francesco-racciatti francesco-racciatti force-pushed the fix-entrypoint-command-override-taskdefinition branch from e2bfc9c to c3fda09 Compare February 22, 2024 08:56
Set the final entrypoint as defined in kilt-definition and use the overridden entrypoint/command from the template when possible. When the template overrides the entrypoint, the command won't be taken from the image.

Signed-off-by: francesco-racciatti <[email protected]>
@francesco-racciatti francesco-racciatti force-pushed the fix-entrypoint-command-override-taskdefinition branch from c3fda09 to 8fe6906 Compare February 22, 2024 09:16
Copy link
Collaborator

@pgcrooks-sysdig pgcrooks-sysdig left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

The test fixtures really help explain the new functionality.

@francesco-racciatti francesco-racciatti merged commit 50da6ac into dev Feb 23, 2024
3 checks passed
@francesco-racciatti francesco-racciatti deleted the fix-entrypoint-command-override-taskdefinition branch February 23, 2024 10:40
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