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

tests/e2e: disable cert-manager.io e2e test #2476

Merged

Conversation

shashankram
Copy link
Member

Description:
There is an ongoing issue with cert-manager.io e2e possibly
timing related, that causes the cert-manager install to fail.
More details can be found in issue #2475.

This change disables the test from running in the CI till
the issue has been addressed. Local runs of the automated
demo work CERT_MANAGER=cert-manager ./demo/run-osm-demo.sh,
although there is a potential timing issue in the existing
demo script as well.

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

Affected area:

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

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

There is an ongoing issue with cert-manager.io e2e possibly
timing related, that causes the cert-manager install to fail.
More details can be found in issue openservicemesh#2475.

This change disables the test from running in the CI till
the issue has been addressed. Local runs of the automated
demo work `CERT_MANAGER=cert-manager ./demo/run-osm-demo.sh`,
although there is a potential timing issue in the existing
demo script as well.

Signed-off-by: Shashank Ram <shashr2204@gmail.com>
@shashankram shashankram requested a review from a team as a code owner February 5, 2021 23:23
@shashankram shashankram merged commit c863662 into openservicemesh:main Feb 5, 2021
@shashankram shashankram deleted the disable-certmanager-e2e branch February 5, 2021 23:43
Copy link
Contributor

@michelleN michelleN left a comment

Choose a reason for hiding this comment

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

We should document somewhere in our docs that there is an ongoing issue and that folks cannot reliably use cert-manager.io

@shashankram
Copy link
Member Author

We should document somewhere in our docs that there is an ongoing issue and that folks cannot reliably use cert-manager.io

This is a test issue, should still be able to use cert-manager.io with OSM if one follows documentation.

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.

4 participants