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

Launch testpmd and trex with an infinite loop entrypoint #92

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

ramperher
Copy link
Collaborator

@ramperher ramperher commented Sep 12, 2024

Not intended to be merged, this is just for experimenting for DPDK Summit 2024 presentation and for grout integration.
The idea of this change is to launch testpmd and trex with an infinite loop as entrypoint, so that we can manually launch the commands to run both tools. For this, we need to remove lifecycle commands and probes to avoid the pod to be restarted.

@dcibot
Copy link

dcibot commented Sep 12, 2024

@dcibot
Copy link

dcibot commented Sep 12, 2024

@ramperher ramperher force-pushed the dpdk-summit-2024 branch 2 times, most recently from 9d64f73 to 8106fa1 Compare September 12, 2024 15:53
@dcibot
Copy link

dcibot commented Sep 12, 2024

@dcibot
Copy link

dcibot commented Sep 13, 2024

@ramperher
Copy link
Collaborator Author

This work is finished, closing this

@ramperher ramperher closed this Sep 19, 2024
@ramperher ramperher reopened this Nov 8, 2024
@ramperher
Copy link
Collaborator Author

Retaking this to test grout integration

@dcibot
Copy link

dcibot commented Nov 8, 2024

@dcibot
Copy link

dcibot commented Nov 11, 2024

@dcibot
Copy link

dcibot commented Nov 14, 2024

@dcibot
Copy link

dcibot commented Nov 19, 2024

@dcibot
Copy link

dcibot commented Nov 19, 2024

ENTRYPOINT ["/usr/local/bin/example-cnf/testpmd-wrapper"]
#ENTRYPOINT ["/usr/local/bin/example-cnf/testpmd-wrapper"]
## Make the container to live forever without executing testpmd-wrapper, we'll run it manually
ENTRYPOINT ["/usr/local/bin/example-cnf/run/infinite.sh"]
Copy link
Collaborator

Choose a reason for hiding this comment

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

why not use sleep infinity instead?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I think I tried in the past and had troubles with that. Anyway I don't intend to merge this change, this is just for troubleshooting

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