Skip to content
This repository has been archived by the owner on Jul 11, 2023. It is now read-only.

ci: run cert-manager scenario with SMI policies enabled #1549

Merged
merged 1 commit into from
Aug 17, 2020

Conversation

shashankram
Copy link
Member

Only the nosplit integration test is meant to run without traffic
split. This change makes the cert-manager scenario testable with
SMI policies similar to the tresor scenario.

Signed-off-by: Shashank Ram shashank08@gmail.com

  • New Functionality [ ]
  • Documentation [ ]
  • Install [ ]
  • Control Plane [ ]
  • CLI Tool [ ]
  • Certificate Management [ ]
  • Networking [ ]
  • Metrics [ ]
  • SMI Policy [ ]
  • Security [ ]
  • Tests / CI System [ ]
  • Other [X]

Please answer the following questions with yes/no.

  • Does this change contain code from or inspired by another project? If so, did you notify the maintainers and provide attribution?
    No

Only the nosplit integration test is meant to run without traffic
split. This change makes the cert-manager scenario testable with
SMI policies similar to the `tresor` scenario.

Signed-off-by: Shashank Ram <shashank08@gmail.com>
@shashankram shashankram marked this pull request as ready for review August 17, 2020 20:09
@shashankram shashankram requested a review from a team as a code owner August 17, 2020 20:09
@shashankram shashankram merged commit 5e71156 into openservicemesh:main Aug 17, 2020
@shashankram shashankram deleted the fix-certmanager-test branch August 17, 2020 20:17
shashankram added a commit to shashankram/osm that referenced this pull request Aug 17, 2020
…servicemesh#1549)"

This reverts commit 5e71156.

Commit 5e71156 accidentally changed
the wrong test.
shashankram added a commit that referenced this pull request Aug 17, 2020
…" (#1551)

This reverts commit 5e71156.

Commit 5e71156 accidentally changed
the wrong test.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants