Skip to content

fixed tests failing due to templates rendering delay #2160

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 3 commits into from
Dec 30, 2022
Merged

Conversation

saeedfazal
Copy link
Contributor

@saeedfazal saeedfazal commented Dec 29, 2022

  • Fixed tests failing due to templates rendering delay
  • Added cluster path parameter to the mvp GitopsTemplate
  • Added leaf-cluster mvp podinfo GitOpsTemplate test
  • Updated CI workflow runs as per refined spec labels
  • Now smoke test run three distinct UI tests (one test per environment i.e. capd, leaf-cluster-tenant, leaf-cluster-gitopstemplate-deploy)

Deploy: https://github.com/weaveworks/weave-gitops-enterprise/actions/runs/3805927176
Nightly: https://github.com/weaveworks/weave-gitops-enterprise/actions/runs/3805928938

@saeedfazal saeedfazal force-pushed the test-refinment branch 2 times, most recently from 8af5182 to 264aa06 Compare December 29, 2022 12:47
@saeedfazal saeedfazal added exclude from release notes Use this label to exclude a PR from the release notes team/pesto labels Dec 29, 2022
@saeedfazal saeedfazal marked this pull request as draft December 29, 2022 13:00
@saeedfazal saeedfazal marked this pull request as ready for review December 29, 2022 13:02
@saeedfazal saeedfazal self-assigned this Dec 29, 2022
@saeedfazal saeedfazal force-pushed the test-refinment branch 4 times, most recently from a454c21 to fa836aa Compare December 29, 2022 20:21
@@ -59,6 +63,7 @@ spec:
namespace: ${{ .params.NAMESPACE }}
annotations:
templates.weave.works/create-request: ""
templates.weave.works/cluster-name: "${{ .params.CLUSTER_PATH }}"
Copy link
Collaborator

Choose a reason for hiding this comment

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

👍 , makes sense, this is here as there seems to be a bug in extracting params from the path component right now.

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.

💯 🥇

@saeedfazal saeedfazal merged commit 36bf72e into main Dec 30, 2022
@saeedfazal saeedfazal deleted the test-refinment branch December 30, 2022 11:24
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