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

chore: Removed makefile and all usages of it #6804

Merged
merged 25 commits into from
Feb 17, 2022
Merged

Conversation

mowies
Copy link
Member

@mowies mowies commented Feb 8, 2022

This PR

  • removes the Makefile from keptn
  • changes all usages of the Makefile to use the scripts underneath directly
  • removes lots of unneeded shell scripts
  • removes the option to set the KUBE_CONSTRAINTS on compile time in the CLI since they are mostly static anyways (this also reduces the places where they need to be adjusted to just 1)
  • makes use of gotestsum in the CLI unit tests

(builds are red because github still tries to use the pipeline from master which doesn't work with this branch)

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>
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>
@codecov
Copy link

codecov bot commented Feb 8, 2022

Codecov Report

Merging #6804 (91de420) into master (5ad04fa) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #6804   +/-   ##
=======================================
  Coverage   57.61%   57.61%           
=======================================
  Files         506      506           
  Lines       28972    28972           
  Branches     1358     1358           
=======================================
  Hits        16692    16692           
  Misses      11068    11068           
  Partials     1212     1212           
Flag Coverage Δ
cli 43.01% <ø> (ø)

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

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>
…ube constraints variables

Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>
Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>
…t value directly

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>
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>
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>
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>
Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>
Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>
@sonarcloud
Copy link

sonarcloud bot commented Feb 14, 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
0.0% 0.0% Duplication

@mowies mowies marked this pull request as ready for review February 14, 2022 14:54
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 changed the title chore: Remove makefile and all usages of it chore: Removed makefile and all usages of it Feb 15, 2022
@mowies mowies merged commit e55355f into master Feb 17, 2022
@mowies mowies deleted the refactor/remove-makefile branch February 17, 2022 12:42
mowies added a commit that referenced this pull request Mar 3, 2022
mowies added a commit that referenced this pull request Mar 3, 2022
(cherry picked from commit e55355f)
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
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants