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

test(openshift): Add demo to OpenShift nightly job #3851

Merged
merged 1 commit into from
Jul 23, 2021

Conversation

ksubrmnn
Copy link
Contributor

Signed-off-by: Kalya Subramanian [email protected]

Description:
Fixes #3805

Adds the demo to the OSM OpenShift nightly job.

After verifying that this passes for a few days, I will create a follow up PR to use Github Actions templates so that the nightly job stays in sync with the PR gates.

Affected area:

Functional Area
New Functionality [ ]
Documentation [ ]
Install [ ]
CLI Tool [ ]
Certificate Management [ ]
Control Plane [ ]
Ingress [ ]
Egress [ ]
Networking [ ]
Observability [ ]
SMI Policy [ ]
Sidecar Injection [ ]
Security [ ]
Upgrade [ ]
Tests [X]
CI System [X]
Demo [ ]
Performance [ ]
Other [ ]

Please answer the following questions with yes/no.

  1. Does this change contain code from or inspired by another project?
    • Did you notify the maintainers and provide attribution?
      No
  2. Is this a breaking change?
    No

@ksubrmnn ksubrmnn requested a review from a team as a code owner July 23, 2021 20:03
@ksubrmnn ksubrmnn requested a review from nshankar13 July 23, 2021 20:03
@@ -26,6 +30,22 @@ jobs:
run: |
make build-osm
go test ./tests/e2e -test.v -ginkgo.v -ginkgo.progress -ginkgo.skip="\bHTTP ingress\b" -ginkgo.skip="\bUpgrade\b" -test.timeout 180m -deployOnOpenShift=true
Copy link
Contributor

Choose a reason for hiding this comment

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

sidebar: is upgrade test passing now with this change? #3835

Copy link
Contributor Author

Choose a reason for hiding this comment

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

We're going to follow the same logic as the PR gates and only run the upgrade tests on the release branch.

I've created a tracking issue here: #3852

@ksubrmnn ksubrmnn merged commit 53e8bc3 into openservicemesh:main Jul 23, 2021
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.

Run OSM automated demo on ARO cluster with GitHub actions
3 participants