Skip to content
This repository was archived by the owner on Sep 17, 2024. It is now read-only.

fix: use staging environment for EPR#1753

Merged
mdelapenya merged 3 commits intoelastic:7.15from
mdelapenya:fix-package-registry-7.15
Nov 2, 2021
Merged

fix: use staging environment for EPR#1753
mdelapenya merged 3 commits intoelastic:7.15from
mdelapenya:fix-package-registry-7.15

Conversation

@mdelapenya
Copy link
Copy Markdown
Contributor

What does this PR do?

It removes the package registry service from the startup list when using the elastic-package provider.

Finally, we are reverting #1750, as it added a local pacakge registry to the compose provider, which is wrong for the ARM flavour of the test suite: there is no ARM Docker image for the package registry.

Why is it important?

Fixes the ARM test suite in 7.15, also adding consistency across branches about what package-registry to use.

Checklist

  • My code follows the style guidelines of this project
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • I have made corresponding change to the default configuration files
  • I have added tests that prove my fix is effective or that my feature works
  • I have run the Unit tests (make unit-test), and they are passing locally
  • I have run the End-2-End tests for the suite I'm working on, and they are passing locally
  • I have noticed new Go dependencies (run make notice in the proper directory)

Related issues

@mdelapenya mdelapenya added Team:Automation Label for the Observability productivity team backport-skip Skip notification from the automated backport with mergify labels Nov 2, 2021
@mdelapenya mdelapenya requested a review from a team November 2, 2021 12:19
@mdelapenya mdelapenya self-assigned this Nov 2, 2021
@mdelapenya
Copy link
Copy Markdown
Contributor Author

Merging, as ARM test suite is green again

@mdelapenya mdelapenya merged commit 3c0c093 into elastic:7.15 Nov 2, 2021
@elasticmachine
Copy link
Copy Markdown
Contributor

💔 Build Failed

the below badges are clickable and redirect to their specific view in the CI or DOCS
Pipeline View Test View Changes Artifacts preview preview

Expand to view the summary

Build stats

  • Start Time: 2021-11-02T12:19:55.794+0000

  • Duration: 46 min 47 sec

  • Commit: 1013ffe

Test stats 🧪

Test Results
Failed 0
Passed 234
Skipped 0
Total 234

Steps errors 1

Expand to view the steps failures

Restore files previously stashed
  • Took 4 min 17 sec . View more details here
  • Description: source

💚 Flaky test report

Tests succeeded.

🤖 GitHub comments

To re-run your PR in the CI, just comment with:

  • /test : Re-trigger the build.

@mdelapenya mdelapenya deleted the fix-package-registry-7.15 branch November 2, 2021 15:59
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

backport-skip Skip notification from the automated backport with mergify Team:Automation Label for the Observability productivity team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants