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

chore: bump Go version #2128

Merged
merged 3 commits into from
Feb 9, 2022

Conversation

mdelapenya
Copy link
Contributor

What does this PR do?

From the Ansible deployment standpoint, it bumps Go version to 7.17

Why is it important?

It was added as part of a backport, that's why we are adding it here

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)

@mdelapenya mdelapenya added Team:Automation Label for the Observability productivity team area:ci Anything related to the CI backport-skip Skip notification from the automated backport with mergify labels Feb 9, 2022
@mdelapenya mdelapenya self-assigned this Feb 9, 2022
@mdelapenya mdelapenya requested review from adam-stokes and a team February 9, 2022 15:11
@elasticmachine
Copy link
Contributor

elasticmachine commented Feb 9, 2022

💔 Tests 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: 2022-02-09T17:35:46.603+0000

  • Duration: 65 min 47 sec

Test stats 🧪

Test Results
Failed 1
Passed 248
Skipped 0
Total 249

Test errors 1

Expand to view the tests failures

Initializing / End-To-End Tests / kubernetes-autodiscover_debian_amd64_elastic-agent / [empty] – TEST-x86_64-kubernetes-autodiscover-75333923-2022-02-09-18:09:09.xml
  • no error details
  • Expand to view the stacktrace

     Test report file /var/lib/jenkins/workspace/PR-2128-3-f777acae-27e5-4a38-9cac-ac438f24ef5f/outputs/18.216.207.219/TEST-x86_64-kubernetes-autodiscover-75333923-2022-02-09-18:09:09.xml was length 0 
    

🐛 Flaky test report

❕ There are test failures but not known flaky tests.

Expand to view the summary

Genuine test errors 1

💔 There are test failures but not known flaky tests, most likely a genuine test failure.

  • Name: Initializing / End-To-End Tests / kubernetes-autodiscover_debian_amd64_elastic-agent / [empty] – TEST-x86_64-kubernetes-autodiscover-75333923-2022-02-09-18:09:09.xml

🤖 GitHub comments

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

  • /test : Re-trigger the build.

We are seeing the following error in tests:

[2022-02-09T15:39:03.406Z] WARN[2022-02-09T15:39:03Z] could not add package to policy because of HTTP code is not 200  elapsedTime=23.117476ms err="<nil>" package"{ apm-68d798ae-aecf-4e63-9354-2cfb53d213a2 Elastic APM default 652bbec0-89be-11ec-88fa-05d7984380d4 true  [{apm true [] map[apm-server:{host localhost:8200}] <nil> <nil>}] { apm Elastic APM 7.16.2}}" response="{\"statusCode\":400,\"error\":\"Bad Request\",\"message\":\"[request body.inputs.0.streams]: expected value of type [array] but got [undefined]\"}" retry=1 statusCode=400
@mdelapenya
Copy link
Contributor Author

YAY! Only the k8s-autodiscover for elastic-agent is failing (reported in #1992)! Merging

@mdelapenya mdelapenya merged commit a37a4cf into elastic:7.17 Feb 9, 2022
@mdelapenya mdelapenya deleted the fix-ansible-deployment-7-17 branch March 9, 2022 06:40
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area:ci Anything related to the CI 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.

3 participants