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

test: Integration Tests on static GKE clusters #6632

Merged
merged 6 commits into from
Feb 1, 2022

Conversation

mowies
Copy link
Member

@mowies mowies commented Jan 19, 2022

This PR

  • migrates the integration test pipeline to use static GKE clusters
  • There are 2 static clusters on GKE
    • keptn-integration-test-1 (GKE 1.19)
    • keptn-integration-test-2 (GKE 1.21)
  • Test runs are separated using k8s namespaces
  • A new GCP token was created to restrict access to GCP and the GKE cluster
  • The kubeconfig to the GKE clusters is now in the GH secrets and not fetched from GCP
  • Istio is now statically installed on GKE clusters and doesn't need to be installed with every test run (the same could be done with Gitea in the future but cleanup could be a challenge)
  • The http server to serve helm chart files is removed in favor of just getting the files locally

Integration test run with static clusters: https://github.com/keptn/keptn/actions/runs/1755712394
(a test is failing but this has nothing to do with the changes from this PR)

@mowies mowies added CI:trigger-build-everything Trigger CI Build: Set BUILD_EVERYTHING=TRUE and removed CI:trigger-build-everything Trigger CI Build: Set BUILD_EVERYTHING=TRUE labels Jan 19, 2022
@mowies mowies changed the title Integration Tests on static GKE clusters test: Integration Tests on static GKE clusters Jan 19, 2022
@codecov
Copy link

codecov bot commented Jan 19, 2022

Codecov Report

Merging #6632 (f9586e1) into master (700895e) will decrease coverage by 0.60%.
The diff coverage is n/a.

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

@@            Coverage Diff             @@
##           master    #6632      +/-   ##
==========================================
- Coverage   57.45%   56.84%   -0.61%     
==========================================
  Files         503      495       -8     
  Lines       28526    28307     -219     
  Branches     1354     1296      -58     
==========================================
- Hits        16389    16091     -298     
- Misses      10955    11010      +55     
- Partials     1182     1206      +24     
Impacted Files Coverage Δ
bridge/server/user/session.ts 28.84% <0.00%> (-48.75%) ⬇️
bridge/server/services/api-service.ts 47.05% <0.00%> (-46.58%) ⬇️
...ent/app/_components/ktb-user/ktb-user.component.ts 66.66% <0.00%> (-33.34%) ⬇️
...notification-bar/ktb-notification-bar.component.ts 80.00% <0.00%> (-20.00%) ⬇️
cli/cmd/install_upgrade_params.go 48.14% <0.00%> (-19.86%) ⬇️
...idge/client/app/_services/notifications.service.ts 86.95% <0.00%> (-13.05%) ⬇️
webhook-service/lib/webhook_config.go 55.00% <0.00%> (-11.67%) ⬇️
shipyard-controller/common/configurationstore.go 77.14% <0.00%> (-11.39%) ⬇️
bridge/server/app.ts 38.54% <0.00%> (-9.24%) ⬇️
cli/cmd/update_project.go 61.29% <0.00%> (-4.23%) ⬇️
... and 68 more
Flag Coverage Δ
api 67.62% <ø> (ø)
approval-service 71.92% <ø> (ø)
bridge-server 49.44% <ø> (-9.69%) ⬇️
cli 41.92% <ø> (-0.63%) ⬇️
configuration-service 8.17% <ø> (-0.05%) ⬇️
distributor 66.92% <ø> (ø)
helm-service 49.04% <ø> (ø)
jmeter-service 13.40% <ø> (ø)
lighthouse-service 74.40% <ø> (ø)
mongodb-datastore 61.24% <ø> (ø)
remediation-service 78.82% <ø> (ø)
secret-service 79.25% <ø> (ø)
statistics-service 59.96% <ø> (ø)
webhook-service 78.85% <ø> (-0.46%) ⬇️

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

@mowies mowies force-pushed the ci/static-integration-test-clusters branch 3 times, most recently from f008c41 to 8c0edd3 Compare January 26, 2022 14:48
Comment on lines +210 to +212
HELM_CHART_NAME=$(ls dist/keptn-installer/keptn*.tgz)
HELM_SERVICE_HELM_CHART_NAME=$(ls dist/keptn-installer/helm*.tgz)
JMETER_SERVICE_HELM_CHART_NAME=$(ls dist/keptn-installer/jmeter*.tgz)
Copy link
Member Author

Choose a reason for hiding this comment

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

take full paths instead of just *.tgz filenames

Comment on lines +269 to +289
export GOOGLE_APPLICATION_CREDENTIALS=~/gcloud-service-key.json
export CLOUDSDK_CORE_DISABLE_PROMPTS=1;

gcloud auth activate-service-account --key-file ~/gcloud-service-key.json
test/utils/gke_create_cluster.sh
test/utils/gke_authenticate_at_cluster.sh

gcloud --quiet config set project "$GCLOUD_PROJECT_NAME"
gcloud --quiet config set container/cluster "$CLUSTER_NAME_NIGHTLY"
gcloud --quiet config set compute/zone "${CLOUDSDK_COMPUTE_ZONE}"

echo "GOOGLE_APPLICATION_CREDENTIALS=${GOOGLE_APPLICATION_CREDENTIALS}" >> $GITHUB_ENV

echo "Setting up kubectl"
echo "Setting KUBECONFIG to $PWD/.kubeconfig"

echo "$GKE_KUBECONFIG" >> "$PWD/.kubeconfig"
chmod 600 "$PWD/.kubeconfig"
export KUBECONFIG="$PWD/.kubeconfig"

echo "KUBECONFIG=${KUBECONFIG}" >> $GITHUB_ENV

echo "Setting kube context..."
kubectl config use-context "$CLUSTER_NAME_NIGHTLY"
Copy link
Member Author

Choose a reason for hiding this comment

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

This sets up gcloud CLI and kubectl with GCP token and kubeconfig for the static test clusters

echo "##[set-output name=CLUSTER_NAME_NIGHTLY;]$(echo ${CLUSTER_NAME_NIGHTLY})"

- name: Install Istio
if: env.CLOUD_PROVIDER != 'minishift-on-GHA' # no need to install istio on minishift
if: env.CLOUD_PROVIDER != 'minishift-on-GHA' && env.CLOUD_PROVIDER != 'GKE' # no need to install istio on minishift or GKE (pre-installed)
Copy link
Member Author

Choose a reason for hiding this comment

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

Istio is now installed statically on the cluster and doesn't need to be installed with every run on GKE anymore

Comment on lines +355 to +359
installer/airgapped/install_keptn.sh "$AIRGAPPED_REGISTRY_URL/" "$HELM_CHART_NAME" "$HELM_SERVICE_HELM_CHART_NAME" "$JMETER_SERVICE_HELM_CHART_NAME"
else
# Install Keptn via CLI
keptn install --platform=${PLATFORM} --namespace=${KEPTN_NAMESPACE} --endpoint-service-type=${KEPTN_SERVICE_TYPE} \
--chart-repo=http://0.0.0.0:8000/${HELM_CHART_NAME} --creds=creds.json --verbose $USE_CASE
--chart-repo=${HELM_CHART_NAME} --creds=creds.json $USE_CASE
Copy link
Member Author

Choose a reason for hiding this comment

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

Charts are installed from file instead of from the web server

Comment on lines -278 to -279
- name: Host helm chart via python http server
run: cd dist/keptn-installer/ && python3 -m http.server &
Copy link
Member Author

Choose a reason for hiding this comment

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

This server is not needed to host the helm chart files. They can just be used directly from the local folder

Comment on lines 493 to 497
helm install helm-service "${HELM_SERVICE_HELM_CHART_NAME}" -n ${{ env.KEPTN_NAMESPACE_PREFIX }}-helm-service --set remoteControlPlane.enabled=true --set remoteControlPlane.api.protocol=http --set remoteControlPlane.api.hostname="${KEPTN_API_HOSTNAME}" --set remoteControlPlane.api.token="${KEPTN_API_TOKEN}" --create-namespace
helm install jmeter-service "${JMETER_SERVICE_HELM_CHART_NAME}" -n ${{ env.KEPTN_NAMESPACE_PREFIX }}-jmeter-service --set remoteControlPlane.enabled=true --set remoteControlPlane.api.protocol=http --set remoteControlPlane.api.hostname="${KEPTN_API_HOSTNAME}" --set remoteControlPlane.api.token="${KEPTN_API_TOKEN}" --create-namespace

helm test jmeter-service -n keptn-jmeter-service
helm test helm-service -n keptn-helm-service
helm test jmeter-service -n ${{ env.KEPTN_NAMESPACE_PREFIX }}-jmeter-service
helm test helm-service -n ${{ env.KEPTN_NAMESPACE_PREFIX }}-helm-service
Copy link
Member Author

Choose a reason for hiding this comment

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

All namespaces during the test run are prefixed with the run number and the attempt number to make them unique on the static clusters

Comment on lines -550 to -555
- name: Uninstall remote execution plane
timeout-minutes: 5
if: env.REMOTE_EXECUTION_PLANE == 'true'
run: |
kubectl delete namespace keptn-helm-service
kubectl delete namespace keptn-jmeter-service
Copy link
Member Author

Choose a reason for hiding this comment

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

Cleanup happens in the Cleanup test namespace step now

@@ -544,32 +565,31 @@ jobs:
mv support-archive/keptn*.zip support-archive/${SUPPORT_ARCHIVE_FILENAME}.zip

- name: Uninstall Keptn
if: always() && env.CLOUD_PROVIDER != 'GKE'
Copy link
Member Author

Choose a reason for hiding this comment

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

Cleanup happens in the Cleanup test namespace step for GKE now

Comment on lines 577 to 597
- name: Cleanup test namespace
if: always() && env.CLOUD_PROVIDER == 'GKE'
run: |
readarray -t namespaces <<< "$(kubectl get namespaces | awk '{ print $1 }' | grep ${{ env.KEPTN_NAMESPACE_PREFIX }})"

if [[ "${{ github.event_name }}" == 'schedule' && "${{ steps.test_aggregated.outcome }}" == 'failure' ]]; then
for namespace in "${namespaces[@]}"; do
echo "Annotating namespace $namespace with Janitor TTL of 3 days..."
kubectl annotate namespace "$namespace" janitor/ttl=3d
done
else
for namespace in "${namespaces[@]}"; do
echo "Deleting namespace $namespace ..."
kubectl delete namespace "$namespace" --wait=false
done
fi
Copy link
Member Author

Choose a reason for hiding this comment

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

This steps takes care of the cleanup on GKE clusters.
It will fetch all namespaces fro the test run and then decide whether they are deleted immediately or keptn for a few days.
If the test run was successful, the namespaces are deleted right away.
If the test run had errors, the namespaces are kept for 3 days and then cleaned up by the kube-janitor operator.

@mowies mowies force-pushed the ci/static-integration-test-clusters branch from ec94369 to dc45578 Compare January 27, 2022 13:41
Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

add some whitespaces here and there

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

remove http server to host helm charts

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

use file paths instead of urls for helm charts during installation

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

restructuring

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

remove debug stuff, re-enable other stuff

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

enable all platforms

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

revert go.sum

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

fix test, revert go.mod

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

ci: Static clusters for integration tests

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

revert debug output

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

only keep test namespaces if there were errors, formatting

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

run cont deliv testcase

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

reduce test suite

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

fix create project method

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

shorten naming prefix

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

remove namespace deletion for now

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

remove namespace deletion for now

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

set namespace prefix explicitely

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

debugging

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

fix some more namespace prefix issues

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

fix helm test commands

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

change prefix so that keptn test namespace is also included in globs

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

use prefix for remote execution plane

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

make use of keptn test prefix variable to prefix all test project names

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

add namespace prefix variable, clean up all namespaces with prefix

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

annotate scheduled runs with janitor ttl to delete them after 3 days

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

use correct kubectl command

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

use 2 platforms again

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

delete other kubeconfig

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

print out keptn config

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

only use one platform for now

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

switch to using helm chart repo instead of self hosted local one

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

fix syntax error

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

omit istio installation on gke clusters

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

force keptn installation

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

dont uninstall istio for debugging

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

reduce helm verbosity, dont delete namespace for debugging

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

always uninstall istio

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

check for istio namespace before prechecking

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

set gcloud credentials path for whole job

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

set kubeconfig in github env so that it is available to all steps in the jobhp

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

remove some debugging, continue with istio debugging

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

reduced context

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

revert self hosted runner stuff

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

use self hosted runner

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

debugging

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

debugging

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

debugging

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

debugging

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

debugging

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

debugging

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

debugging

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

remove unneeded files, add file content to pipeline step, add debug output

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

debugging

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

debugging

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

debugging

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

write kubeconfig to file, then set KUBECONFIG variable

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

remove get-credentials calls

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

use new gcp token, use new kubeconfig

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

more gcloud cli debugging

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

debugging

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

change namespace name to run number instead of run id

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

remove unneeded test platforms for now

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

switch to using unique namespace for every integration test run, always delete namespace after tests

Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>
@mowies mowies force-pushed the ci/static-integration-test-clusters branch from dc45578 to 2be545d Compare January 27, 2022 13:45
@mowies mowies marked this pull request as ready for review January 27, 2022 13:50
@mowies mowies removed the request for review from christian-kreuzberger-dtx January 27, 2022 13:50
Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>
.github/workflows/integration_tests.yml Show resolved Hide resolved
.github/workflows/integration_tests.yml Outdated Show resolved Hide resolved
test/go-tests/test_utils.go Outdated Show resolved Hide resolved
test/go-tests/test_resourceservice.go Outdated Show resolved Hide resolved
test/go-tests/test_resourceservice.go Outdated Show resolved Hide resolved
mowies and others added 3 commits January 31, 2022 16:16
Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

Co-authored-by: Giovanni Liva <giovanni.liva@dynatrace.com>
… the keptn test namespace

Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>
Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>
odubajDT
odubajDT previously approved these changes Jan 31, 2022
Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>
@sonarcloud
Copy link

sonarcloud bot commented Feb 1, 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 merged commit ca9bebe into master Feb 1, 2022
@mowies mowies deleted the ci/static-integration-test-clusters branch February 1, 2022 08:27
@mowies
Copy link
Member Author

mowies commented Feb 2, 2022

This fixes #5322

mowies added a commit that referenced this pull request Mar 3, 2022
Co-authored-by: Giovanni Liva <giovanni.liva@dynatrace.com>
Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>
mowies added a commit that referenced this pull request Mar 3, 2022
Co-authored-by: Giovanni Liva <giovanni.liva@dynatrace.com>
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

4 participants