Skip to content

[updatecli][githubrelease] Bump version to 1.19.8#2475

Merged
cmacknz merged 4 commits intomainfrom
updatecli_6cbcaba217a0e93f926b1764128b2e5c4165c13ed502e6b8c3e45cfc2d7cba2d
Apr 18, 2023
Merged

[updatecli][githubrelease] Bump version to 1.19.8#2475
cmacknz merged 4 commits intomainfrom
updatecli_6cbcaba217a0e93f926b1764128b2e5c4165c13ed502e6b8c3e45cfc2d7cba2d

Conversation

@ghost
Copy link
Copy Markdown

@ghost ghost commented Apr 8, 2023

[updatecli][githubrelease] Bump version to 1.19.8

It requires the version to be bumped first in golang-crossbuild project, then a new release will be added to:
https://github.com/elastic/golang-crossbuild/releases/tag/v1.19.8.
Otherwise it will fail until the docker images are available.

Report

Source:
	✔ [gomod] Get version in go.mod format(shell)
	✔ [latestGoVersion] Get Latest Go Release(githubrelease)
	✔ [minor] Get minor version in .go-version(shell)


Condition:
	✔ [dockerTag] Is docker image golang:1.19.8 published(dockerimage)
	✔ [goDefaultVersion-check] Check if defined golang version differs(shell)

Target:
	✔ [update-dockerfiles] Update from dockerfiles(file)
	✔ [update-go-version] Update .go-version(file)
	✔ [update-golang.ci] Update .golangci.yml(file)
	✔ [update-gomod] Update go.mod(file)
	✔ [update-version.asciidoc] Update version.asciidoc(file)

Changelog

Click to expand
no GitHub Release found for go1.19.8 on "https://github.com/golang/go"


Remark

This pull request was automatically created using Updatecli.

Please report any issues with this tool here

apmmachine added 4 commits April 8, 2023 20:01
... /elastic-agent/.go-version"

Made with ❤️️ by updatecli
... /elastic-agent/Dockerfile"

Made with ❤️️ by updatecli
... /elastic-agent/version/docs/version.asciidoc"

Made with ❤️️ by updatecli
... /elastic-agent/.golangci.yml"

Made with ❤️️ by updatecli
@ghost ghost self-requested a review as a code owner April 8, 2023 20:01
@ghost ghost requested review from AndersonQ and pchila and removed request for a team April 8, 2023 20:01
@ghost ghost added the backport-skip label Apr 8, 2023
@mergify mergify Bot assigned ghost Apr 8, 2023
@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: 2023-04-08T20:01:53.174+0000

  • Duration: 22 min 51 sec

Test stats 🧪

Test Results
Failed 0
Passed 5423
Skipped 23
Total 5446

💚 Flaky test report

Tests succeeded.

🤖 GitHub comments

Expand to view the GitHub comments

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

  • /test : Re-trigger the build.

  • /package : Generate the packages.

  • run integration tests : Run the Elastic Agent Integration tests.

  • run end-to-end tests : Generate the packages and run the E2E Tests.

  • run elasticsearch-ci/docs : Re-trigger the docs validation. (use unformatted text in the comment!)

@elasticmachine
Copy link
Copy Markdown
Contributor

🌐 Coverage report

Name Metrics % (covered/total) Diff
Packages 98.507% (66/67) 👍
Files 69.432% (159/229) 👍
Classes 68.65% (300/437) 👍
Methods 53.968% (918/1701) 👍
Lines 39.034% (10273/26318) 👍 0.004
Conditionals 100.0% (0/0) 💚

@cmacknz cmacknz merged commit 732ad1f into main Apr 18, 2023
@cmacknz cmacknz deleted the updatecli_6cbcaba217a0e93f926b1764128b2e5c4165c13ed502e6b8c3e45cfc2d7cba2d branch April 18, 2023 17:37
@cmacknz cmacknz added the backport-v8.7.0 Automated backport with mergify label Apr 18, 2023
mergify Bot pushed a commit that referenced this pull request Apr 18, 2023
* chore: Updated the content of the file "/tmp/updatecli/github/elastic...

... /elastic-agent/.go-version"

Made with ❤️️ by updatecli

* chore: Updated the content of the file "/tmp/updatecli/github/elastic...

... /elastic-agent/Dockerfile"

Made with ❤️️ by updatecli

* chore: Updated the content of the file "/tmp/updatecli/github/elastic...

... /elastic-agent/version/docs/version.asciidoc"

Made with ❤️️ by updatecli

* chore: Updated the content of the file "/tmp/updatecli/github/elastic...

... /elastic-agent/.golangci.yml"

Made with ❤️️ by updatecli

---------

Co-authored-by: apmmachine <apmmachine@users.noreply.github.com>
(cherry picked from commit 732ad1f)
@ycombinator
Copy link
Copy Markdown
Contributor

ycombinator commented Apr 19, 2023

Unfortunately, this PR being merged before elastic/golang-crossbuild#282 is causing Agent builds to be broken on main, for example: https://fleet-ci.elastic.co/job/elastic-agent/job/elastic-agent-mbp/job/main/908/console:

10:51:27  >> golangCrossBuild: Building for linux/arm64
10:51:27  >> buildGoDaemon: Building for linux/arm64
10:51:27  Unable to find image 'docker.elastic.co/beats-dev/golang-crossbuild:1.19.8-arm' locally
10:51:27  Unable to find image 'docker.elastic.co/beats-dev/golang-crossbuild:1.19.8-arm' locally
10:51:28  docker: Error response from daemon: manifest for docker.elastic.co/beats-dev/golang-crossbuild:1.19.8-arm not found: manifest unknown: manifest unknown.
10:51:28  See 'docker run --help'.
10:51:28  docker: Error response from daemon: manifest for docker.elastic.co/beats-dev/golang-crossbuild:1.19.8-arm not found: manifest unknown: manifest unknown.
10:51:28  See 'docker run --help'.
10:51:28  package ran for 1m23.959849782s
10:51:28  Error: running "docker run --env EXEC_UID=1147 --env EXEC_GID=1147 -v /var/lib/jenkins/workspace/tic-agent_elastic-agent-mbp_main/pkg/mod:/go/pkg/mod:ro --rm --env GOFLAGS=-mod=readonly --env MAGEFILE_VERBOSE= --env MAGEFILE_TIMEOUT= --env SNAPSHOT=true --env DEV=true --env EXTERNAL=true -v /var/lib/jenkins/workspace/tic-agent_elastic-agent-mbp_main/src/github.com/elastic/elastic-agent:/go/src/github.com/elastic/elastic-agent -w /go/src/github.com/elastic/elastic-agent docker.elastic.co/beats-dev/golang-crossbuild:1.19.8-arm --build-cmd build/mage-linux-arm64 buildGoDaemon --platforms linux/arm64" failed with exit code 125
10:51:28  running "docker run --env EXEC_UID=1147 --env EXEC_GID=1147 -v /var/lib/jenkins/workspace/tic-agent_elastic-agent-mbp_main/pkg/mod:/go/pkg/mod:ro --rm --env GOFLAGS=-mod=readonly --env MAGEFILE_VERBOSE= --env MAGEFILE_TIMEOUT= --env SNAPSHOT=true --env DEV=true --env EXTERNAL=true -v /var/lib/jenkins/workspace/tic-agent_elastic-agent-mbp_main/src/github.com/elastic/elastic-agent:/go/src/github.com/elastic/elastic-agent -w /go/src/github.com/elastic/elastic-agent docker.elastic.co/beats-dev/golang-crossbuild:1.19.8-arm --build-cmd build/mage-linux-arm64 golangCrossBuild --platforms linux/arm64" failed with exit code 125

Should we temporarily revert this PR merge, until elastic/golang-crossbuild#282 is merged first?

@cmacknz
Copy link
Copy Markdown
Member

cmacknz commented Apr 19, 2023

Ugh, yes we should revert this.

We also need to look into why the build "succeeded" without any use of the cross build image #2475 (comment)

cmacknz added a commit to cmacknz/elastic-agent that referenced this pull request Apr 19, 2023
@cmacknz
Copy link
Copy Markdown
Member

cmacknz commented Apr 19, 2023

#2515

@ycombinator
Copy link
Copy Markdown
Contributor

ycombinator commented Apr 19, 2023

Comparing the "successful" build from this PR (https://fleet-ci.elastic.co/blue/organizations/jenkins/elastic-agent%2Felastic-agent-mbp/detail/PR-2512/1/pipeline/909) and the failing build on main (https://fleet-ci.elastic.co/blue/organizations/jenkins/elastic-agent%2Felastic-agent-mbp/detail/main/908/pipeline/822), we can see that the packaging error happens in the failing build when mage package ironbank is called. For whatever reason, in the "successful" build, the Package stage never gets around to calling mage package ironbank.

I randomly sampled some other "successful" PR builds, viz:

In every case the Package stage never gets around to calling mage package (I think the ironbank target is irrelevant to this situation). Instead, the Package stage just has a couple of these log entries for it:

[2023-04-11T11:33:24.083Z] [INFO] githubApiCall: just been called.

So we need to figure out what's different in the Package stage's configuration between PR builds and main builds and why.

@ycombinator
Copy link
Copy Markdown
Contributor

ycombinator commented Apr 19, 2023

So we need to figure out what's different in the Package stage's configuration between PR builds and main builds and why.

I'm not too familiar with the Jenkins CI config DSL but I think this is why the Package stage might not be running on PR builds:

not { changeRequest() }

I'll put up a test PR with that line removed and see if it makes the Package stage run.

@ycombinator
Copy link
Copy Markdown
Contributor

ycombinator commented Apr 19, 2023

I'm not too familiar with the Jenkins CI config DSL but I think this is why the Package stage might not be running on PR builds:

not { changeRequest() }

I'll put up a test PR with that line removed and see if it makes the Package stage run.

Test PR is up: #2516. And it looks like CI is running the Package stage on that PR and, as of this writing, failing as would be expected: https://fleet-ci.elastic.co/blue/organizations/jenkins/elastic-agent%2Felastic-agent-mbp/detail/PR-2516/1/pipeline/656.

cmacknz added a commit that referenced this pull request Apr 19, 2023
cmacknz pushed a commit that referenced this pull request Apr 24, 2023
* chore: Updated the content of the file "/tmp/updatecli/github/elastic...

... /elastic-agent/.go-version"

Made with ❤️️ by updatecli

* chore: Updated the content of the file "/tmp/updatecli/github/elastic...

... /elastic-agent/Dockerfile"

Made with ❤️️ by updatecli

* chore: Updated the content of the file "/tmp/updatecli/github/elastic...

... /elastic-agent/version/docs/version.asciidoc"

Made with ❤️️ by updatecli

* chore: Updated the content of the file "/tmp/updatecli/github/elastic...

... /elastic-agent/.golangci.yml"

Made with ❤️️ by updatecli

---------

Co-authored-by: apmmachine <apmmachine@users.noreply.github.com>
(cherry picked from commit 732ad1f)

Co-authored-by: apmmachine <58790750+apmmachine@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport-skip backport-v8.7.0 Automated backport with mergify

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants