Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bump elastic stack version automation #1081

Merged
merged 20 commits into from
Apr 21, 2021

Conversation

v1v
Copy link
Member

@v1v v1v commented Apr 20, 2021

What does this PR do?

Pipeline automation to bump the elastic stack version in the given list of repositories. This will create a PR if there are new changes to be applied.

Why is it important?

No more issues when consuming latest snapshots.

Related issues

Closes #ISSUE

Tests

Manually tested with dry-run mode

image

image

v1v and others added 10 commits March 18, 2021 21:14
* 'master' of github.com:elastic/apm-pipeline-library: (40 commits)
  Fix and simplify buildStatus step (#1066)
  Add benchmark reporter image to build pipeline (#1065)
  docs: update CHANGELOG.md
  [maven-release-plugin] prepare for next development iteration
  [maven-release-plugin] prepare release v1.1.207
  feat: stage status cache (#1061)
  feat: step hasCommentAuthorWritePermissions (#1064)
  ARM workers are now ephemeral (#1062)
  docs: update metricbeat step docs (#1057)
  Update beats-ci workers for the test-infra validation (#1058)
  feat: allow to remove padding (#1059)
  mergify: priority is now a pro feature (#1060)
  docs: update CHANGELOG.md
  [maven-release-plugin] prepare for next development iteration
  [maven-release-plugin] prepare release v1.1.206
  chore(deps): bump groovy-eclipse-batch from 3.0.7-02 to 3.0.7-03 (#1045)
  chore: bump Elastic Stack versions (#1052)
  Add GCE JCasC (#1053)
  feat: metricbeat step (#1050)
  chore: refactor filebeat step (#1049)
  ...
* 'master' of github.com:elastic/apm-pipeline-library:
  test: add Ubuntu 20.04 to the tests (#1068)
  docs: update CHANGELOG.md
  [maven-release-plugin] prepare for next development iteration
  [maven-release-plugin] prepare release v1.1.208
  Disable unique ephemeral worker by default (#1067)
t checkout -b# the commit.
* 'master' of github.com:elastic/apm-pipeline-library:
  fix: use toJSON instead readJSON (#1071)
  test-infra: add curl validation on windows (#1069)
  CI: skip using ubuntu label in the beats test-infra (#1070)
* 'master' of github.com:elastic/apm-pipeline-library:
  chore(deps-dev): bump mockito-core from 3.8.0 to 3.9.0 (#1073)
* 'master' of github.com:elastic/apm-pipeline-library:
  docs: update CHANGELOG.md
  [maven-release-plugin] prepare for next development iteration
  [maven-release-plugin] prepare release v1.1.209
  security: disable codecov bash updater
@apmmachine
Copy link
Contributor

apmmachine commented Apr 20, 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

Expand to view the summary

Build stats

  • Build Cause: Pull request #1081 updated

  • Start Time: 2021-04-21T12:36:56.106+0000

  • Duration: 35 min 56 sec

  • Commit: e314890

Test stats 🧪

Test Results
Failed 0
Passed 1115
Skipped 2
Total 1117

Trends 🧪

Image of Build Times

Image of Tests

💚 Flaky test report

Tests succeeded.

Expand to view the summary

Test stats 🧪

Test Results
Failed 0
Passed 1115
Skipped 2
Total 1117

@v1v v1v marked this pull request as ready for review April 20, 2021 15:13
@v1v v1v requested a review from a team April 20, 2021 15:14
@v1v v1v self-assigned this Apr 20, 2021
def generateSteps(Map args = [:]) {
def projects = readYaml(file: '.ci/.bump-stack-version.yml')
def parallelTasks = [:]
projects['projects'].each { p ->
Copy link
Contributor

Choose a reason for hiding this comment

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

an alternative implementation

matrix(
          agent: 'ubuntu-20',
          axes: [
            axis('JOB_PROJECT', projects ),
            axis('JOB_BRANCHES', branches )
          ]
        ){
        ...
        }

Copy link
Member Author

Choose a reason for hiding this comment

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

Cool, I'll give a go!

Copy link
Member Author

Choose a reason for hiding this comment

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

matrix does not work well with Field variables :/

10:22:33  ERROR: Pull Requests failed
10:22:33  Also:   groovy.lang.MissingPropertyException: No such property: latestVersions for class: groovy.lang.Binding
10:22:33  Also:   groovy.lang.MissingPropertyException: No such property: latestVersions for class: groovy.lang.Binding
10:22:33  groovy.lang.MissingPropertyException: No such property: latestVersions for class: groovy.lang.Binding
10:22:33  	at groovy.lang.Binding.getVariable(Binding.java:63)
10:22:33  	at org.jenkinsci.plugins.scriptsecurity.sandbox.groovy.SandboxInterceptor.onGetProperty(SandboxInterceptor.java:270)
10:22:33  	at org.kohsuke.groovy.sandbox.impl.Checker$7.call(Checker.java:353)
10:22:33  	at org.kohsuke.groovy.sandbox.impl.Checker.checkedGetProperty(Checker.java:357)
10:22:33  	at org.kohsuke.groovy.sandbox.impl.Checker.checkedGetProperty(Checker.java:333)
10:22:33  	at org.kohsuke.groovy.sandbox.impl.Checker.checkedGetProperty(Checker.java:333)
10:22:33  	at com.cloudbees.groovy.cps.sandbox.SandboxInvoker.getProperty(SandboxInvoker.java:29)
10:22:33  	at com.cloudbees.groovy.cps.impl.PropertyAccessBlock.rawGet(PropertyAccessBlock.java:20)
10:22:33  	at WorkflowScript.generateSteps(WorkflowScript:60)
10:22:33  	at matrix.call(matrix.groovy:35)

So I'll not use it for the time being (9eacd58 is the commit with the proposal)

Copy link
Member Author

Choose a reason for hiding this comment

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

I said nothing! I had a wrong variable name :/

image

v1v added 6 commits April 21, 2021 10:24
This reverts commit 9eacd58.
…v/apm-pipeline-library into feature/bump-stack-version-automation

* 'feature/bump-stack-version-automation' of github.com:v1v/apm-pipeline-library:
  Enable stage filter argument for the beatsStages (#1076)
@mergify mergify bot merged commit 951ac82 into elastic:master Apr 21, 2021
@v1v v1v deleted the feature/bump-stack-version-automation branch April 21, 2021 13:12
v1v added a commit that referenced this pull request Apr 21, 2021
…notify-message-template

* upstream/master:
  Fix build status and adjust Vault (#1074)
  Remove super-linter since it takes 20 minutes (#1083)
  Bump elastic stack version automation (#1081)
  feat: step to check if a docker image exists (#1077)
  docs: update CHANGELOG.md
  [maven-release-plugin] prepare for next development iteration
  [maven-release-plugin] prepare release v1.1.210
  Enable stage filter argument for the beatsStages (#1076)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants