Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Run observability tests in upgrade tests #13318

Merged
merged 17 commits into from Mar 4, 2022

Conversation

shorim
Copy link
Contributor

@shorim shorim commented Feb 8, 2022

Description

Changes proposed in this pull request:

  • Run Loki tests as well in monitoring suite
  • Run monitoring tests in upgrade scenarios
  • Skip monitoring tests in upgrade scenarios that do major upgrade 1 to 2

Related issue(s)

@shorim shorim added area/monitoring Issues or PRs related to the monitoring module (deprecated) area/logging Issues or PRs related to the logging module (deprecated) do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. labels Feb 8, 2022
@shorim shorim added this to the 2.1 milestone Feb 8, 2022
@kyma-bot kyma-bot added size/S Denotes a PR that changes 10-29 lines, ignoring generated files. and removed do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. labels Feb 8, 2022
@kyma-bot kyma-bot added size/M Denotes a PR that changes 30-99 lines, ignoring generated files. and removed size/S Denotes a PR that changes 10-29 lines, ignoring generated files. labels Feb 8, 2022
@shorim shorim linked an issue Feb 9, 2022 that may be closed by this pull request
@kyma-bot kyma-bot added size/L Denotes a PR that changes 100-499 lines, ignoring generated files. and removed size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels Feb 9, 2022
@lindnerby lindnerby self-assigned this Feb 10, 2022
@lindnerby lindnerby changed the title Observability upgrade tests Run observability tests in upgrade tests Feb 28, 2022
@halamix2
Copy link
Member

halamix2 commented Mar 2, 2022

/retest

1 similar comment
@rakesh-garimella
Copy link
Contributor

/retest

@rakesh-garimella
Copy link
Contributor

/retest

3 similar comments
@rakesh-garimella
Copy link
Contributor

/retest

@rakesh-garimella
Copy link
Contributor

/retest

@rakesh-garimella
Copy link
Contributor

/retest

@kyma-bot kyma-bot added the lgtm Looks good to me! label Mar 4, 2022
@kyma-bot kyma-bot merged commit ef42293 into kyma-project:main Mar 4, 2022
cnvergence pushed a commit to cnvergence/kyma that referenced this pull request Mar 8, 2022
* add logging test to upgrade tests

* move monitoring tests

* update import of monitoringTests

* add monitoring tests to upgrade tests

* missing trailing comma

* missing trailing comma

* remove the export of 3-monitoring module

* remove scrape pool prefixes

* Skip monitoring tests for Kyma 1 versions

* Try Kyma major version env

* Add migration guide for upgrade from version 2.0.4 to newer

* Move migration guide to separate PR

* Delete assets

* Make false as string as boolean evaluation does not work

Co-authored-by: Benjamin Lindner <benjamin.lindner@sap.com>
Co-authored-by: Rakesh Garimella <rakesh.garimella@sap.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/logging Issues or PRs related to the logging module (deprecated) area/monitoring Issues or PRs related to the monitoring module (deprecated) lgtm Looks good to me! size/M Denotes a PR that changes 30-99 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Observability tests to be integrated into standard test scenario
5 participants