Skip to content

Conversation

saeedfazal
Copy link
Contributor

  • Fixed nightly test failing due to orphan resources from previous day run
  • Add capi provider cleanup for cluster reset
  • Add explicit CAPI_PROVIDER environment variable to install capi provider to management cluster
  • Update the acceptance user guide

@saeedfazal saeedfazal added the exclude from release notes Use this label to exclude a PR from the release notes label May 19, 2022
@saeedfazal saeedfazal self-assigned this May 19, 2022
@saeedfazal saeedfazal force-pushed the cleanup-nightly-fix branch from 18d2eb8 to 9e34f9f Compare May 19, 2022 15:14
Copy link
Member

@sarataha sarataha left a comment

Choose a reason for hiding this comment

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

LGTM 💯

@@ -849,6 +849,7 @@ func DescribeTemplates(gitopsTestRunner GitopsTestRunner) {
}

setParameterValues(createPage, paramSection)
pages.ScrollWindow(webDriver, 0, 4000)
Copy link
Collaborator

Choose a reason for hiding this comment

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

Interesting, do we see failures without this?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

yes, rarely one of the nightly test fail being not able to locate the 'Preview PR' button. I found scrolling explicitly fixes the issue.

@saeedfazal saeedfazal merged commit 093b4bf into main May 20, 2022
@saeedfazal saeedfazal deleted the cleanup-nightly-fix branch May 20, 2022 08:42
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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants