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

Additions to CCNP/CNP Pipeline #500

Open
wants to merge 95 commits into
base: main
Choose a base branch
from

Conversation

karina-ranadive
Copy link
Contributor

@karina-ranadive karina-ranadive commented Jan 29, 2025

Hello,

I had a few cilium metrics for measurements for ccnp & cnp scale testing. I also adjusted the cnp and ccnp policies being deployed. I also made it possible for the user to enter a number for repeats. I also changed the stage name.

@karina-ranadive karina-ranadive changed the title Added Cilium Metrics & Adjusted CCNP/CNP Policies Additions to CCNP/CNP Pipeline Jan 29, 2025
@karina-ranadive karina-ranadive force-pushed the karanadive/additionstoccnp-cnpPipeline branch 2 times, most recently from 8faeaec to e0cea7f Compare January 30, 2025 02:35
Copy link
Contributor Author

@karina-ranadive karina-ranadive Jan 30, 2025

Choose a reason for hiding this comment

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

I changed "$REPEATS" to ${REPEATS:-1} in collect.yml & execute.yml just in case in pipeline user does not input a value for repeats just so there are no errors in case

@karina-ranadive karina-ranadive force-pushed the karanadive/additionstoccnp-cnpPipeline branch from 7915cd6 to c3ed3ac Compare January 31, 2025 15:42
@karina-ranadive karina-ranadive marked this pull request as ready for review January 31, 2025 17:07
@karina-ranadive karina-ranadive force-pushed the karanadive/additionstoccnp-cnpPipeline branch from f3faeae to 1b9a1a9 Compare February 25, 2025 16:04
@karina-ranadive karina-ranadive force-pushed the karanadive/additionstoccnp-cnpPipeline branch from 649e382 to 2b93082 Compare March 10, 2025 15:19
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