Skip to content

Set Elastic licence type for APM server Beats update job#24115

Closed
kvch wants to merge 1 commit intoelastic:masterfrom
kvch:fix-apm-licence-text
Closed

Set Elastic licence type for APM server Beats update job#24115
kvch wants to merge 1 commit intoelastic:masterfrom
kvch:fix-apm-licence-text

Conversation

@kvch
Copy link
Copy Markdown
Contributor

@kvch kvch commented Feb 18, 2021

What does this PR do?

Adds a new licence override temporarily when testing if Beats can be updated in APM Server.

Why is it important?

We get the following error in the Beats update job:

Failed to detect licences: failed to detect licence type of /var/lib/jenkins/workspace/Beats_apm-beats-update_master/src/github.com/elastic/beats-local from /var/lib/jenkins/workspace/Beats_apm-beats-update_master/src/github.com/elastic/beats-local/LICENSE.txt:
failed to detect licence type of /var/lib/jenkins/workspace/Beats_apm-beats-update_master/src/github.com/elastic/beats-local/LICENSE.txt

@kvch kvch requested a review from kuisathaverat February 18, 2021 18:02
@kvch kvch requested a review from a team as a code owner February 18, 2021 18:02
@botelastic botelastic Bot added needs_team Indicates that the issue/PR needs a Team:* label Team:Automation Label for the Observability productivity team and removed needs_team Indicates that the issue/PR needs a Team:* label labels Feb 18, 2021
Copy link
Copy Markdown
Member

@jsoriano jsoriano left a comment

Choose a reason for hiding this comment

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

Thanks!

@v1v
Copy link
Copy Markdown
Member

v1v commented Feb 18, 2021

I've just triggered -> https://beats-ci.elastic.co/job/Beats/job/apm-beats-update/view/change-requests/job/PR-24115/3/

I guess that's the job

@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

Expand to view the summary

Build stats

  • Build Cause: Pull request #24115 opened

  • Start Time: 2021-02-18T18:03:06.609+0000

  • Duration: 12 min 22 sec

  • Commit: c025d9c

Trends 🧪

Image of Build Times

❕ Flaky test report

No test was executed to be analysed.

@v1v
Copy link
Copy Markdown
Member

v1v commented Feb 18, 2021

/run apm-beats-update

@kvch
Copy link
Copy Markdown
Contributor Author

kvch commented Feb 19, 2021

Closing in favour of #24117

@kvch kvch closed this Feb 19, 2021
@kvch
Copy link
Copy Markdown
Contributor Author

kvch commented Feb 19, 2021

Reopening as the PR mentioned above does not help.

@kvch
Copy link
Copy Markdown
Contributor Author

kvch commented Feb 19, 2021

New PR: #24122

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Team:Automation Label for the Observability productivity team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants