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

fix: do not use a local package-registry#1756

Merged
mdelapenya merged 3 commits intoelastic:masterfrom
mdelapenya:revert-local-package-registry
Nov 2, 2021
Merged

fix: do not use a local package-registry#1756
mdelapenya merged 3 commits intoelastic:masterfrom
mdelapenya:revert-local-package-registry

Conversation

@mdelapenya
Copy link
Copy Markdown
Contributor

What does this PR do?

This PR reverts #1743, also forcing the elastic-package provider to use the staging environment of EPR, not starting a local package-registry

Why is it important?

Keep behaviours in sync with all branches

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 the backport-skip Skip notification from the automated backport with mergify label Nov 2, 2021
@mdelapenya mdelapenya self-assigned this Nov 2, 2021
@mdelapenya mdelapenya requested a review from a team November 2, 2021 14:47
@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-02T14:48:10.858+0000

  • Duration: 12 min 7 sec

  • Commit: 0c9f7b5

Test stats 🧪

Test Results
Failed 0
Passed 131
Skipped 0
Total 131

Steps errors 2

Expand to view the steps failures

Install precommit hooks
  • Took 0 min 1 sec . View more details here
  • Description: export PATH=/var/lib/jenkins/workspace/2e-tests_e2e-testing-mbp_PR-1756/bin:/var/lib/jenkins/workspace/2e-tests_e2e-testing-mbp_PR-1756/bin:/var/lib/jenkins/workspace/2e-tests_e2e-testing-mbp_PR-1756/.gvm/versions/go1.16.3.linux.amd64/bin:/var/lib/jenkins/workspace/2e-tests_e2e-testing-mbp_PR-1756/src/github.com/elastic/e2e-testing/bin:/usr/local/bin:/usr/bin:/bin:/usr/sbin:/sbin:/var/lib/jenkins/workspace/2e-tests_e2e-testing-mbp_PR-1756/bin:/var/lib/jenkins/workspace/2e-tests_e2e-testing-mbp_PR-1756/src/github.com/elastic/e2e-testing/.ci/scripts ## Install with the hooks therefore ~/.cache/pre-commit will be created with the repos pre-commit install --install-hooks
Install precommit hooks
  • Took 0 min 2 sec . View more details here
  • Description: export PATH=/var/lib/jenkins/workspace/2e-tests_e2e-testing-mbp_PR-1756/bin:/var/lib/jenkins/workspace/2e-tests_e2e-testing-mbp_PR-1756/bin:/var/lib/jenkins/workspace/2e-tests_e2e-testing-mbp_PR-1756/.gvm/versions/go1.16.3.linux.amd64/bin:/var/lib/jenkins/workspace/2e-tests_e2e-testing-mbp_PR-1756/src/github.com/elastic/e2e-testing/bin:/usr/local/bin:/usr/bin:/bin:/usr/sbin:/sbin:/var/lib/jenkins/workspace/2e-tests_e2e-testing-mbp_PR-1756/bin:/var/lib/jenkins/workspace/2e-tests_e2e-testing-mbp_PR-1756/src/github.com/elastic/e2e-testing/.ci/scripts ## Install with the hooks therefore ~/.cache/pre-commit will be created with the repos pre-commit install --install-hooks

💚 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 added the Team:Automation Label for the Observability productivity team label Nov 2, 2021
@mdelapenya
Copy link
Copy Markdown
Contributor Author

Merging as the ARM test (docker-compose based) suite is back to green, and AMD (elastic-package based) is green too

@mdelapenya mdelapenya merged commit 5bdd72f into elastic:master Nov 2, 2021
@mdelapenya mdelapenya deleted the revert-local-package-registry branch November 2, 2021 15:58
@elasticmachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

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-02T15:25:17.933+0000

  • Duration: 43 min 13 sec

  • Commit: 5e19c26

Test stats 🧪

Test Results
Failed 0
Passed 248
Skipped 0
Total 248

💚 Flaky test report

Tests succeeded.

🤖 GitHub comments

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

  • /test : Re-trigger the build.

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