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

Test upgrade from previous version to latest version in CI #1831

Open
kpollich opened this issue May 14, 2024 · 3 comments
Open

Test upgrade from previous version to latest version in CI #1831

kpollich opened this issue May 14, 2024 · 3 comments
Assignees
Labels
Team:Ecosystem Label for the Packages Ecosystem team technnical debt

Comments

@kpollich
Copy link
Member

We currently don't have automated test coverage in CI for integration upgrades. It could be helpful to run automated tests against the stack for each package that assert an upgrade from the previous published version to the latest version is successful. This would catch cases where both versions of a package are considered valid, but backwards compatibility is broken between versions.

Relates to #579

@kpollich kpollich added Team:Ecosystem Label for the Packages Ecosystem team technnical debt labels May 14, 2024
@kpollich kpollich assigned mrodm and unassigned jsoriano Jun 25, 2024
@andrewkroh
Copy link
Member

I came here to open a new issue, but found this one already opened. Here is what I was going to write.

Test package upgrades

There are bugs that could be detected if we could make elastic-package execute the package upgrade process. Roughly what I would like to see tested is this scenario:

  1. Install the latest version from the EPR.
  2. Do something to create indices. I think we need indices to catch mapping upgrade problems.
  3. Install (upgrade) to package present in the pull request.
  4. Run the system test.

I think this would help catch mapping and transform upgrade problems.

Related

@jsoriano
Copy link
Member

This could be done in the context of the compatibility tests proposed in #579.

@cmacknz
Copy link
Member

cmacknz commented Sep 10, 2024

+1, either as part of #579 or this issue we need this. We have shipped bugs this test would have caught.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Team:Ecosystem Label for the Packages Ecosystem team technnical debt
Projects
None yet
Development

No branches or pull requests

5 participants