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

global: remove pointer to containerSpec #185

Merged
merged 1 commit into from
Sep 4, 2024

Conversation

sebrandon1
Copy link
Collaborator

@sebrandon1 sebrandon1 commented Sep 3, 2024

Depends-on: openshift-kni/eco-goinfra#638

Modifies the calls to the deployment pkg in the eco-goinfra library to be more consistent with the daemonset pkg.

Bonus: I added the depends-on github action to the makefile.yml to allow for bringing in the dependent pull request as a requirement.

@sebrandon1 sebrandon1 self-assigned this Sep 3, 2024
@sebrandon1 sebrandon1 force-pushed the deployment_pointer_removal branch 4 times, most recently from 5bfb474 to 31a343d Compare September 3, 2024 18:27
@sebrandon1 sebrandon1 changed the title Remove pointer to containerSpec global: remove pointer to containerSpec Sep 4, 2024
Copy link
Collaborator

@kononovn kononovn left a comment

Choose a reason for hiding this comment

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

/lgtm

@kononovn kononovn merged commit c578ebc into openshift-kni:main Sep 4, 2024
1 check passed
@sebrandon1 sebrandon1 deleted the deployment_pointer_removal branch September 4, 2024 15:53
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.

2 participants