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

[meta] upgrade test (#907) [7.x] #920

Merged
merged 2 commits into from
Nov 20, 2020
Merged

Conversation

jmlrt
Copy link
Member

@jmlrt jmlrt commented Nov 20, 2020

Backports the following commits to 7.x:

This PR is adding a new upgrade test for every chart.

This test is deploying the oldest **release** version of charts which can be upgraded to latest version, then upgrading using unreleased charts in the repo.

The goal is to detect when some PR is introducing a breaking change.

- Note 1: An upgrade test is already existing for Elasticsearch but had been removed from master branch because upgrade from 7.x to Elasticsearch 8.0.0-SNAPSHOT used by master branch isn't working. The new test allows overriding the Docker image version used during upgrade so we can still test upgrade on master but use the latest 7.x Docker image instead of 8.0.0-SNAPSHOT.

* Note 2:  Metricbeat chart introduced a breaking change in elastic#516 and can be upgraded from any previous version currently, so Metricbeat test is commented and should be uncommented after 7.10.0 release so we can check if new PR are introducing new breaking changes.

* Note 3: Currently charts upgrade are only tested with default values and so cover a small part of charts codebase, we should update it to test optional resources (ie: enable service or ingress, ...).

Fix elastic#396
@jmlrt jmlrt requested a review from a team November 20, 2020 11:46
@jmlrt jmlrt merged commit ba1b165 into elastic:7.x Nov 20, 2020
@jmlrt jmlrt deleted the backport/7.x/pr-907 branch November 20, 2020 15:32
@jmlrt jmlrt changed the title [7.x] [meta] upgrade test (#907) [meta] upgrade test (#907) [7.x] Nov 23, 2020
@jmlrt jmlrt mentioned this pull request Feb 8, 2021
@jmlrt jmlrt mentioned this pull request Feb 18, 2021
This was referenced Mar 15, 2021
@jmlrt jmlrt mentioned this pull request May 25, 2021
@jmlrt jmlrt mentioned this pull request Mar 8, 2022
@jmlrt jmlrt mentioned this pull request Apr 21, 2022
This was referenced Sep 14, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant