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

chore: bump elastic-package to v0.32.1 #1959

Merged
merged 5 commits into from
Jan 10, 2022

Conversation

mdelapenya
Copy link
Contributor

  • chore: bump elastic-package
  • fix: use agent complete as fleet-server
  • chore: bump go version to 1.17

What does this PR do?

It bumps elastic-package to v0.32.1 (its latest version), and that forces us to bump the Go version to 1.17

On the same hand, because the changes in the elastic-package (see elastic/elastic-package#637) are updating the base image for fleet-server, to use the elastic-agent complete flavour (including Synthetics and NodeJS runtime), we are using the very same image in the Docker Compose provisioner of this project.

Why is it important?

Keep dependencies up-to-date

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)

How to test this PR locally

Using docker provider:

TAGS="integrations" TIMEOUT_FACTOR=3 LOG_LEVEL=TRACE DEVELOPER_MODE=true ELASTIC_APM_ACTIVE=false PROVIDER=docker make -C e2e/_suites/fleet functional-test

Using elastic-package provider:

TAGS="integrations" TIMEOUT_FACTOR=3 LOG_LEVEL=TRACE DEVELOPER_MODE=true ELASTIC_APM_ACTIVE=false PROVIDER=elastic-package make -C e2e/_suites/fleet functional-test

Related issues

Follow-ups

We may need to enable dependabot for tracking changes in the elastic-package dependency 🤔

@mdelapenya mdelapenya added the backport-v8.0.0 Automated backport with mergify label Dec 29, 2021
@mdelapenya mdelapenya self-assigned this Dec 29, 2021
@mdelapenya mdelapenya requested a review from a team December 29, 2021 14:55
@elasticmachine
Copy link
Contributor

elasticmachine commented Dec 29, 2021

💚 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: 2022-01-04T08:06:17.955+0000

  • Duration: 33 min 25 sec

  • Commit: 010b37b

Test stats 🧪

Test Results
Failed 0
Passed 216
Skipped 0
Total 216

💚 Flaky test report

Tests succeeded.

🤖 GitHub comments

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

  • /test : Re-trigger the build.

@mdelapenya
Copy link
Contributor Author

/test

@@ -35,7 +35,7 @@
- role: andrewrothstein.kind
- role: gantsign.golang
vars:
golang_version: 1.16.3
golang_version: 1.17
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We need a way to pass .go-version file to here

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@adam-stokes what do you think about having a Make goal reading .go-version to install Go using Andrew Kroh's https://github.com/andrewkroh/gvm? Therefore, we won't duplicate Go versions in multiple places:

eval "$(gvm $(cat .go-version))"

@mdelapenya mdelapenya requested a review from adam-stokes January 4, 2022 08:06
@mdelapenya
Copy link
Contributor Author

@adam-stokes we are only using the elastic package provider in one test suite, right? Should we apply it everywhere?

@adam-stokes
Copy link
Contributor

@adam-stokes we are only using the elastic package provider in one test suite, right? Should we apply it everywhere?

I think that's a good idea

@mdelapenya mdelapenya merged commit 686c531 into elastic:main Jan 10, 2022
mergify bot pushed a commit that referenced this pull request Jan 10, 2022
* chore: bump elastic-package

* fix: use agent complete as fleet-server

* chore: bump go version to 1.17

It's required by elastic-package dependencies

* fix: match Go version in Ansible

(cherry picked from commit 686c531)

# Conflicts:
#	cli/config/compose/profiles/fleet/docker-compose.yml
mdelapenya added a commit to mdelapenya/e2e-testing that referenced this pull request Jan 26, 2022
* main: (45 commits)
  feat: add CentOS 8 support (elastic#2034)
  fix: set default region for AWS cli (elastic#2053)
  chore: use Ansible's built-in replace instead of sed (elastic#2048)
  chore: split stack configuration and start into two tasks (elastic#2044)
  feat: enable SSH access to users for debugging cloud instances (elastic#2001)
  fix: use the right branch for 7.17 backports (elastic#2025)
  SLES15 enablement (elastic#2007)
  chore: bump stale agent for main (elastic#2014)
  Update `fetchBeatsBinary` to be reused in elastic-agent-poc (elastic#1984)
  chore: add resiliency when provisioning the stack (elastic#1990)
  chore: bump elastic-package to v0.32.1 (elastic#1959)
  feat: export Fetch&Download methods in the /pkg directory (elastic#1943)
  bump stack version 8.1.0-dbc834fd (elastic#1948)
  bump stack version 8.1.0-76902d39 (elastic#1946)
  chore: retire 7.15 adding 7.17 (elastic#1938)
  ci: use withAPMEnv (elastic#1917)
  Update main branch (elastic#1928)
  bump stack version 8.1.0-befff95a (elastic#1929)
  chore: properly evaluate how tests are skipped on CI when checking modified files (elastic#1924)
  bump stack version 8.1.0-60bffc32 (elastic#1921)
  ...
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
backport-v8.0.0 Automated backport with mergify
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants