-
Couldn't load subscription status.
- Fork 25.6k
[Test] Fix test022InstallPluginsFromLocalArchive #131353
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] Fix test022InstallPluginsFromLocalArchive #131353
Conversation
The number of plugin archives changed and triggered the assertion. Changing the assertion to check for `analysis-icu` plugin archive instead of asserting on number of archives. This will always fail when new plugin is added (like microsoft-graph-authz). Resolves elastic#116866
|
Pinging @elastic/es-delivery (Team:Delivery) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice improvement. LGTM.
as in elastic#131353 Changing the assertion to check for analysis-icu plugin archive instead of asserting on number of archives. This will always fail when new plugin is added (like microsoft-graph-authz).
* Standardize on docker image arch classifier (#130643) * The number of plugin archives changed and triggered the assertion as in #131353 Changing the assertion to check for analysis-icu plugin archive instead of asserting on number of archives. This will always fail when new plugin is added (like microsoft-graph-authz). --------- Co-authored-by: Mariusz Józala <[email protected]> Co-authored-by: Mariusz Jozala <[email protected]>
💔 Backport failed
You can use sqren/backport to manually backport by running |
The number of plugin archives changed and triggered the assertion.
Changing the assertion to check for
analysis-icuplugin archiveinstead of asserting on number of archives. This will always fail
when new plugin is added (like microsoft-graph-authz).
Resolves #116866