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

ci: Fix intergation tests taking master artifacts from bot commits #6670

Merged
merged 6 commits into from
Jan 25, 2022

Conversation

mowies
Copy link
Member

@mowies mowies commented Jan 25, 2022

This PR

  • enhances the jq command that filters the GH actions runs on master to find the latest successful build.
  • Now, the filter takes bot builds into account, since those don't push any images which in turn makes the integration tests fail during keptn installation

Integration test run where the correct Run ID was extracted from previous master runs:
https://github.com/keptn/keptn/runs/4934107939?check_suite_focus=true#step:8:36

Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>
@mowies mowies added the CI:trigger-build-everything Trigger CI Build: Set BUILD_EVERYTHING=TRUE label Jan 25, 2022
Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>
@codecov
Copy link

codecov bot commented Jan 25, 2022

Codecov Report

Merging #6670 (d31230c) into master (3a2c944) will decrease coverage by 0.01%.
The diff coverage is n/a.

❗ Current head d31230c differs from pull request most recent head 94e9387. Consider uploading reports for the commit 94e9387 to get more accurate results

@@            Coverage Diff             @@
##           master    #6670      +/-   ##
==========================================
- Coverage   56.72%   56.71%   -0.02%     
==========================================
  Files         496      496              
  Lines       28391    28391              
  Branches     1301     1301              
==========================================
- Hits        16106    16103       -3     
- Misses      11072    11074       +2     
- Partials     1213     1214       +1     
Impacted Files Coverage Δ
distributor/pkg/lib/controlplane/uniformlog.go 71.87% <0.00%> (-4.69%) ⬇️
Flag Coverage Δ
api 67.62% <ø> (ø)
approval-service 71.92% <ø> (ø)
bridge-server 49.44% <ø> (ø)
cli 42.12% <ø> (ø)
configuration-service 8.22% <ø> (ø)
distributor 66.59% <ø> (-0.33%) ⬇️
helm-service 49.04% <ø> (ø)
jmeter-service 13.40% <ø> (ø)
lighthouse-service 74.40% <ø> (ø)
mongodb-datastore 61.24% <ø> (ø)
remediation-service 78.82% <ø> (ø)
resource-service 83.36% <ø> (ø)
secret-service 79.25% <ø> (ø)
shipyard-controller 57.26% <ø> (ø)
statistics-service 59.96% <ø> (ø)
webhook-service 78.85% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

@mowies mowies marked this pull request as draft January 25, 2022 08:11
@mowies mowies removed the request for review from christian-kreuzberger-dtx January 25, 2022 08:11
Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>
@mowies mowies removed the CI:trigger-build-everything Trigger CI Build: Set BUILD_EVERYTHING=TRUE label Jan 25, 2022
Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>
Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>
Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>
@mowies mowies marked this pull request as ready for review January 25, 2022 08:46
@sonarcloud
Copy link

sonarcloud bot commented Jan 25, 2022

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@mowies mowies requested a review from a team January 25, 2022 08:48
@oleg-nenashev oleg-nenashev added the automation CI, Nightlies, Build Jobs label Jan 25, 2022
Copy link
Contributor

@odubajDT odubajDT left a comment

Choose a reason for hiding this comment

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

LGTM

@mowies mowies merged commit 650e2ca into master Jan 25, 2022
@mowies mowies deleted the ci/fix-integration-tests branch January 25, 2022 08:51
mowies added a commit that referenced this pull request Mar 3, 2022
…6670)

Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>
mowies added a commit that referenced this pull request Mar 3, 2022
…6670)

Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>
bacherfl pushed a commit that referenced this pull request Mar 3, 2022
* ci: Add pipeline to nightly check docker image digests (#6598)

Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* ci: Minor pipeline improvements (#6613)

Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* ci: Only push docker images if not from a forked repo (#6618)

Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* ci: Fix digest checker pipeline (#6619)

Co-authored-by: Giovanni Liva <giovanni.liva@dynatrace.com>
Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* ci: Fix CI pipeline not pushing master images to registry (#6634)

Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* feat: Release helm charts on GitHub pages (#6559)

BREAKING CHANGES: The Keptn Helm charts are now served from the [keptn/helm-charts](https://github.com/keptn/helm-charts) repository through GitHub pages. They are accessible by using https://charts.keptn.sh as a helm chart repository.

Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* ci: Remove unneeded pipelines and config files (#6642)

Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* ci: Fix master builds not pushing docker images (#6643)

Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* ci: Fix master builds not pushing docker images (#6659)

Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* ci: Fix intergation tests taking master artifacts from bot commits (#6670)

Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* test: Integration Tests on static GKE clusters (#6632)

Co-authored-by: Giovanni Liva <giovanni.liva@dynatrace.com>
Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* chore: Add k8s resource stats to release notes (#6718)

Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* ci: Fix dev helm charts not having a unique image name (#6816)

Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* ci: Disable manual triggers of the CI pipeline (#6819)

Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* ci: Always use datetimed version for integration tests (#6820)

Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* ci: Ensure that build matrix runs as far as possible (#6875)

Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* ci: Only upload dev helm chart for non-fork PRs (#6763)

(cherry picked from commit 134d487)
Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* ci: Push dev images to dev dockerhub registry, adjust helm charts (#6857)

(cherry picked from commit 59a7762)
Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* chore: Removed makefile and all usages of it (#6804)

(cherry picked from commit e55355f)
Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* ci: Fix cleanup in integration tests to also include clusterroles and clusterrolebindings (#6931)

(cherry picked from commit 3dd71e5)
Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* ci: Fix release pipeline not uploading helm charts correctly (#6944)

(cherry picked from commit 0609c7d)
Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* ci: Fix integration test pipeline not starting if there were 404ing CI pipeline runs in the source branch (#6966)

(cherry picked from commit 23c4c37)
Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* ci: Fix resource cleanup in integration tests when resource arrays are empty (#6971)

(cherry picked from commit 2abb21c)
Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* Remove backport artifact

Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

Co-authored-by: Giovanni Liva <giovanni.liva@dynatrace.com>
bacherfl added a commit that referenced this pull request Mar 3, 2022
…stributor/integration version (#7046)

* fix(shipyard-controller): Do not overwrite existing subscriptions in case of version update

Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>

* added assertion to check if subscriptions are still there after version upgrade

Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>

* added assertion to check if subscriptions are still there after version upgrade

Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>

* fixed typo

Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>

* fixed sonar warnings

Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>

* added comment

Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>

* pr review

Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>

* added unit test for repo implementation

Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>

* ci: Add pipeline to nightly check docker image digests (#6598)

Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* ci: Minor pipeline improvements (#6613)

Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* ci: Only push docker images if not from a forked repo (#6618)

Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* ci: Fix digest checker pipeline (#6619)

Co-authored-by: Giovanni Liva <giovanni.liva@dynatrace.com>
Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* ci: Fix CI pipeline not pushing master images to registry (#6634)

Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* feat: Release helm charts on GitHub pages (#6559)

BREAKING CHANGES: The Keptn Helm charts are now served from the [keptn/helm-charts](https://github.com/keptn/helm-charts) repository through GitHub pages. They are accessible by using https://charts.keptn.sh as a helm chart repository.

Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* ci: Remove unneeded pipelines and config files (#6642)

Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* ci: Fix master builds not pushing docker images (#6643)

Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* ci: Fix master builds not pushing docker images (#6659)

Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* ci: Fix intergation tests taking master artifacts from bot commits (#6670)

Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* test: Integration Tests on static GKE clusters (#6632)

Co-authored-by: Giovanni Liva <giovanni.liva@dynatrace.com>
Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* chore: Add k8s resource stats to release notes (#6718)

Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* ci: Fix dev helm charts not having a unique image name (#6816)

Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* ci: Disable manual triggers of the CI pipeline (#6819)

Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* ci: Always use datetimed version for integration tests (#6820)

Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* ci: Ensure that build matrix runs as far as possible (#6875)

Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* ci: Only upload dev helm chart for non-fork PRs (#6763)

(cherry picked from commit 134d487)
Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* ci: Push dev images to dev dockerhub registry, adjust helm charts (#6857)

(cherry picked from commit 59a7762)
Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* chore: Removed makefile and all usages of it (#6804)

(cherry picked from commit e55355f)
Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* ci: Fix cleanup in integration tests to also include clusterroles and clusterrolebindings (#6931)

(cherry picked from commit 3dd71e5)
Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* ci: Fix release pipeline not uploading helm charts correctly (#6944)

(cherry picked from commit 0609c7d)
Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* ci: Fix integration test pipeline not starting if there were 404ing CI pipeline runs in the source branch (#6966)

(cherry picked from commit 23c4c37)
Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* ci: Fix resource cleanup in integration tests when resource arrays are empty (#6971)

(cherry picked from commit 2abb21c)
Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* added comment

Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>

* fix(shipyard-controller): Do not overwrite existing subscriptions in case of version update

Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>

* added assertion to check if subscriptions are still there after version upgrade

Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>

* added assertion to check if subscriptions are still there after version upgrade

Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>

* fixed typo

Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>

* fixed sonar warnings

Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>

* added comment

Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>

* pr review

Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>

* added unit test for repo implementation

Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>

* added comment

Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>

Co-authored-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>
Co-authored-by: Giovanni Liva <giovanni.liva@dynatrace.com>
bacherfl added a commit that referenced this pull request Mar 3, 2022
…stributor/integration version (#7046)

* fix(shipyard-controller): Do not overwrite existing subscriptions in case of version update

Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>

* added assertion to check if subscriptions are still there after version upgrade

Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>

* added assertion to check if subscriptions are still there after version upgrade

Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>

* fixed typo

Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>

* fixed sonar warnings

Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>

* added comment

Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>

* pr review

Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>

* added unit test for repo implementation

Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>

* ci: Add pipeline to nightly check docker image digests (#6598)

Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* ci: Minor pipeline improvements (#6613)

Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* ci: Only push docker images if not from a forked repo (#6618)

Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* ci: Fix digest checker pipeline (#6619)

Co-authored-by: Giovanni Liva <giovanni.liva@dynatrace.com>
Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* ci: Fix CI pipeline not pushing master images to registry (#6634)

Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* feat: Release helm charts on GitHub pages (#6559)

BREAKING CHANGES: The Keptn Helm charts are now served from the [keptn/helm-charts](https://github.com/keptn/helm-charts) repository through GitHub pages. They are accessible by using https://charts.keptn.sh as a helm chart repository.

Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* ci: Remove unneeded pipelines and config files (#6642)

Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* ci: Fix master builds not pushing docker images (#6643)

Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* ci: Fix master builds not pushing docker images (#6659)

Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* ci: Fix intergation tests taking master artifacts from bot commits (#6670)

Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* test: Integration Tests on static GKE clusters (#6632)

Co-authored-by: Giovanni Liva <giovanni.liva@dynatrace.com>
Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* chore: Add k8s resource stats to release notes (#6718)

Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* ci: Fix dev helm charts not having a unique image name (#6816)

Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* ci: Disable manual triggers of the CI pipeline (#6819)

Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* ci: Always use datetimed version for integration tests (#6820)

Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* ci: Ensure that build matrix runs as far as possible (#6875)

Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* ci: Only upload dev helm chart for non-fork PRs (#6763)

(cherry picked from commit 134d487)
Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* ci: Push dev images to dev dockerhub registry, adjust helm charts (#6857)

(cherry picked from commit 59a7762)
Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* chore: Removed makefile and all usages of it (#6804)

(cherry picked from commit e55355f)
Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* ci: Fix cleanup in integration tests to also include clusterroles and clusterrolebindings (#6931)

(cherry picked from commit 3dd71e5)
Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* ci: Fix release pipeline not uploading helm charts correctly (#6944)

(cherry picked from commit 0609c7d)
Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* ci: Fix integration test pipeline not starting if there were 404ing CI pipeline runs in the source branch (#6966)

(cherry picked from commit 23c4c37)
Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* ci: Fix resource cleanup in integration tests when resource arrays are empty (#6971)

(cherry picked from commit 2abb21c)
Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* added comment

Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>

* fix(shipyard-controller): Do not overwrite existing subscriptions in case of version update

Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>

* added assertion to check if subscriptions are still there after version upgrade

Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>

* added assertion to check if subscriptions are still there after version upgrade

Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>

* fixed typo

Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>

* fixed sonar warnings

Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>

* added comment

Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>

* pr review

Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>

* added unit test for repo implementation

Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>

* added comment

Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>

Co-authored-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>
Co-authored-by: Giovanni Liva <giovanni.liva@dynatrace.com>
(cherry picked from commit a5a62ee)
bacherfl added a commit that referenced this pull request Mar 3, 2022
…stributor/integration version (#7046)

* fix(shipyard-controller): Do not overwrite existing subscriptions in case of version update

Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>

* added assertion to check if subscriptions are still there after version upgrade

Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>

* added assertion to check if subscriptions are still there after version upgrade

Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>

* fixed typo

Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>

* fixed sonar warnings

Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>

* added comment

Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>

* pr review

Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>

* added unit test for repo implementation

Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>

* ci: Add pipeline to nightly check docker image digests (#6598)

Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* ci: Minor pipeline improvements (#6613)

Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* ci: Only push docker images if not from a forked repo (#6618)

Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* ci: Fix digest checker pipeline (#6619)

Co-authored-by: Giovanni Liva <giovanni.liva@dynatrace.com>
Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* ci: Fix CI pipeline not pushing master images to registry (#6634)

Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* feat: Release helm charts on GitHub pages (#6559)

BREAKING CHANGES: The Keptn Helm charts are now served from the [keptn/helm-charts](https://github.com/keptn/helm-charts) repository through GitHub pages. They are accessible by using https://charts.keptn.sh as a helm chart repository.

Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* ci: Remove unneeded pipelines and config files (#6642)

Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* ci: Fix master builds not pushing docker images (#6643)

Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* ci: Fix master builds not pushing docker images (#6659)

Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* ci: Fix intergation tests taking master artifacts from bot commits (#6670)

Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* test: Integration Tests on static GKE clusters (#6632)

Co-authored-by: Giovanni Liva <giovanni.liva@dynatrace.com>
Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* chore: Add k8s resource stats to release notes (#6718)

Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* ci: Fix dev helm charts not having a unique image name (#6816)

Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* ci: Disable manual triggers of the CI pipeline (#6819)

Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* ci: Always use datetimed version for integration tests (#6820)

Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* ci: Ensure that build matrix runs as far as possible (#6875)

Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* ci: Only upload dev helm chart for non-fork PRs (#6763)

(cherry picked from commit 134d487)
Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* ci: Push dev images to dev dockerhub registry, adjust helm charts (#6857)

(cherry picked from commit 59a7762)
Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* chore: Removed makefile and all usages of it (#6804)

(cherry picked from commit e55355f)
Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* ci: Fix cleanup in integration tests to also include clusterroles and clusterrolebindings (#6931)

(cherry picked from commit 3dd71e5)
Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* ci: Fix release pipeline not uploading helm charts correctly (#6944)

(cherry picked from commit 0609c7d)
Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* ci: Fix integration test pipeline not starting if there were 404ing CI pipeline runs in the source branch (#6966)

(cherry picked from commit 23c4c37)
Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* ci: Fix resource cleanup in integration tests when resource arrays are empty (#6971)

(cherry picked from commit 2abb21c)
Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* added comment

Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>

* fix(shipyard-controller): Do not overwrite existing subscriptions in case of version update

Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>

* added assertion to check if subscriptions are still there after version upgrade

Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>

* added assertion to check if subscriptions are still there after version upgrade

Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>

* fixed typo

Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>

* fixed sonar warnings

Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>

* added comment

Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>

* pr review

Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>

* added unit test for repo implementation

Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>

* added comment

Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>

Co-authored-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>
Co-authored-by: Giovanni Liva <giovanni.liva@dynatrace.com>
(cherry picked from commit a5a62ee)
bacherfl added a commit that referenced this pull request Mar 4, 2022
…stributor/integration version (#7046) (#7059)

* fix(shipyard-controller): Do not overwrite existing subscriptions in case of version update

Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>

* added assertion to check if subscriptions are still there after version upgrade

Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>

* added assertion to check if subscriptions are still there after version upgrade

Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>

* fixed typo

Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>

* fixed sonar warnings

Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>

* added comment

Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>

* pr review

Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>

* added unit test for repo implementation

Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>

* ci: Add pipeline to nightly check docker image digests (#6598)

Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* ci: Minor pipeline improvements (#6613)

Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* ci: Only push docker images if not from a forked repo (#6618)

Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* ci: Fix digest checker pipeline (#6619)

Co-authored-by: Giovanni Liva <giovanni.liva@dynatrace.com>
Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* ci: Fix CI pipeline not pushing master images to registry (#6634)

Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* feat: Release helm charts on GitHub pages (#6559)

BREAKING CHANGES: The Keptn Helm charts are now served from the [keptn/helm-charts](https://github.com/keptn/helm-charts) repository through GitHub pages. They are accessible by using https://charts.keptn.sh as a helm chart repository.

Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* ci: Remove unneeded pipelines and config files (#6642)

Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* ci: Fix master builds not pushing docker images (#6643)

Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* ci: Fix master builds not pushing docker images (#6659)

Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* ci: Fix intergation tests taking master artifacts from bot commits (#6670)

Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* test: Integration Tests on static GKE clusters (#6632)

Co-authored-by: Giovanni Liva <giovanni.liva@dynatrace.com>
Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* chore: Add k8s resource stats to release notes (#6718)

Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* ci: Fix dev helm charts not having a unique image name (#6816)

Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* ci: Disable manual triggers of the CI pipeline (#6819)

Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* ci: Always use datetimed version for integration tests (#6820)

Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* ci: Ensure that build matrix runs as far as possible (#6875)

Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* ci: Only upload dev helm chart for non-fork PRs (#6763)

(cherry picked from commit 134d487)
Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* ci: Push dev images to dev dockerhub registry, adjust helm charts (#6857)

(cherry picked from commit 59a7762)
Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* chore: Removed makefile and all usages of it (#6804)

(cherry picked from commit e55355f)
Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* ci: Fix cleanup in integration tests to also include clusterroles and clusterrolebindings (#6931)

(cherry picked from commit 3dd71e5)
Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* ci: Fix release pipeline not uploading helm charts correctly (#6944)

(cherry picked from commit 0609c7d)
Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* ci: Fix integration test pipeline not starting if there were 404ing CI pipeline runs in the source branch (#6966)

(cherry picked from commit 23c4c37)
Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* ci: Fix resource cleanup in integration tests when resource arrays are empty (#6971)

(cherry picked from commit 2abb21c)
Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* added comment

Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>

* fix(shipyard-controller): Do not overwrite existing subscriptions in case of version update

Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>

* added assertion to check if subscriptions are still there after version upgrade

Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>

* added assertion to check if subscriptions are still there after version upgrade

Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>

* fixed typo

Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>

* fixed sonar warnings

Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>

* added comment

Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>

* pr review

Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>

* added unit test for repo implementation

Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>

* added comment

Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>

Co-authored-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>
Co-authored-by: Giovanni Liva <giovanni.liva@dynatrace.com>
(cherry picked from commit a5a62ee)
bacherfl added a commit that referenced this pull request Mar 7, 2022
…stributor/integration version (#7046) (#7058)

* fix(shipyard-controller): Do not overwrite existing subscriptions in case of version update

Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>

* added assertion to check if subscriptions are still there after version upgrade

Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>

* added assertion to check if subscriptions are still there after version upgrade

Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>

* fixed typo

Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>

* fixed sonar warnings

Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>

* added comment

Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>

* pr review

Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>

* added unit test for repo implementation

Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>

* ci: Add pipeline to nightly check docker image digests (#6598)

Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* ci: Minor pipeline improvements (#6613)

Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* ci: Only push docker images if not from a forked repo (#6618)

Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* ci: Fix digest checker pipeline (#6619)

Co-authored-by: Giovanni Liva <giovanni.liva@dynatrace.com>
Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* ci: Fix CI pipeline not pushing master images to registry (#6634)

Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* feat: Release helm charts on GitHub pages (#6559)

BREAKING CHANGES: The Keptn Helm charts are now served from the [keptn/helm-charts](https://github.com/keptn/helm-charts) repository through GitHub pages. They are accessible by using https://charts.keptn.sh as a helm chart repository.

Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* ci: Remove unneeded pipelines and config files (#6642)

Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* ci: Fix master builds not pushing docker images (#6643)

Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* ci: Fix master builds not pushing docker images (#6659)

Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* ci: Fix intergation tests taking master artifacts from bot commits (#6670)

Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* test: Integration Tests on static GKE clusters (#6632)

Co-authored-by: Giovanni Liva <giovanni.liva@dynatrace.com>
Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* chore: Add k8s resource stats to release notes (#6718)

Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* ci: Fix dev helm charts not having a unique image name (#6816)

Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* ci: Disable manual triggers of the CI pipeline (#6819)

Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* ci: Always use datetimed version for integration tests (#6820)

Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* ci: Ensure that build matrix runs as far as possible (#6875)

Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* ci: Only upload dev helm chart for non-fork PRs (#6763)

(cherry picked from commit 134d487)
Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* ci: Push dev images to dev dockerhub registry, adjust helm charts (#6857)

(cherry picked from commit 59a7762)
Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* chore: Removed makefile and all usages of it (#6804)

(cherry picked from commit e55355f)
Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* ci: Fix cleanup in integration tests to also include clusterroles and clusterrolebindings (#6931)

(cherry picked from commit 3dd71e5)
Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* ci: Fix release pipeline not uploading helm charts correctly (#6944)

(cherry picked from commit 0609c7d)
Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* ci: Fix integration test pipeline not starting if there were 404ing CI pipeline runs in the source branch (#6966)

(cherry picked from commit 23c4c37)
Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* ci: Fix resource cleanup in integration tests when resource arrays are empty (#6971)

(cherry picked from commit 2abb21c)
Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

* added comment

Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>

* fix(shipyard-controller): Do not overwrite existing subscriptions in case of version update

Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>

* added assertion to check if subscriptions are still there after version upgrade

Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>

* added assertion to check if subscriptions are still there after version upgrade

Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>

* fixed typo

Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>

* fixed sonar warnings

Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>

* added comment

Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>

* pr review

Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>

* added unit test for repo implementation

Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>

* added comment

Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>

Co-authored-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>
Co-authored-by: Giovanni Liva <giovanni.liva@dynatrace.com>
(cherry picked from commit a5a62ee)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
automation CI, Nightlies, Build Jobs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants