Skip to content

Conversation

saeedfazal
Copy link
Contributor

@saeedfazal saeedfazal commented Jul 14, 2022

  • Moved smoke tests to main
  • Moved acceptances tests to nightly
  • Nightly acceptance tests push the test results notification to slack #github-feed-pesto channel

Screenshot 2022-07-25 at 09 55 57

- The workflows can be triggered manually for any branch including nightly

Manually triggered workflow jobs

Deploy: https://github.com/weaveworks/weave-gitops-enterprise/actions/runs/2731133310
Nightly: https://github.com/weaveworks/weave-gitops-enterprise/actions/runs/2731134729

@saeedfazal saeedfazal added the exclude from release notes Use this label to exclude a PR from the release notes label Jul 14, 2022
@saeedfazal saeedfazal self-assigned this Jul 14, 2022
@saeedfazal saeedfazal force-pushed the schedule-acceptance-tests branch 3 times, most recently from b921d00 to 5147b0b Compare July 14, 2022 11:26
@saeedfazal saeedfazal force-pushed the schedule-acceptance-tests branch 15 times, most recently from bdeb1c4 to 99ba024 Compare July 25, 2022 09:40
@saeedfazal saeedfazal force-pushed the schedule-acceptance-tests branch from baf739e to f670a94 Compare July 25, 2022 16:29
@@ -87,7 +87,7 @@ func DescribeClusters(gitopsTestRunner GitopsTestRunner) {
var _ = Describe("Multi-Cluster Control Plane Clusters", func() {

Context("[UI] When no leaf cluster is connected", func() {
It("Verify connected cluster dashboard shows only management cluster", Label("integration"), func() {
It("Verify connected cluster dashboard shows only management cluster", Label("integration", "saeed"), func() {
Copy link
Collaborator

Choose a reason for hiding this comment

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

🤔 ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

good catch, will address it, left over typo from trials

Copy link
Collaborator

@foot foot left a comment

Choose a reason for hiding this comment

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

Nice one! 🎉 Excited to test it out.

@saeedfazal saeedfazal merged commit 1ce4788 into main Jul 26, 2022
@saeedfazal saeedfazal deleted the schedule-acceptance-tests branch July 26, 2022 08:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
exclude from release notes Use this label to exclude a PR from the release notes team/pesto
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants