Skip to content

Conversation

@mdelapenya
Copy link
Contributor

@mdelapenya mdelapenya commented Nov 8, 2021

What does this PR do?

It fixes how array lengths are calculated when retrieving the files to be uploaded to GC storage.

Intead of using .length() as a method, which is incorrect, it uses .length as an attribute. As an alternative we could use .size() method.

Why is it important?

We are seeing this failure in the pipeline:

groovy.lang.MissingMethodException: No signature of method: [Lorg.jenkinsci.plugins.pipeline.utility.steps.fs.FileWrapper;.length() is applicable for argument types: () values: []
Possible solutions: last(), init(), getAt(groovy.lang.EmptyRange), getAt(java.util.Collection), getAt(groovy.lang.Range), getAt(java.lang.Integer)

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 added an entry in CHANGELOG.next.asciidoc or CHANGELOG-developer.next.asciidoc.

Related issues

@mdelapenya mdelapenya requested a review from a team as a code owner November 8, 2021 19:22
@mdelapenya mdelapenya added Team:Automation Label for the Observability productivity team backport-v7.15.0 Automated backport with mergify backport-v7.16.0 Automated backport with mergify backport-v8.0.0 Automated backport with mergify labels Nov 8, 2021
@mdelapenya mdelapenya self-assigned this Nov 8, 2021
@botelastic botelastic bot added needs_team Indicates that the issue/PR needs a Team:* label and removed needs_team Indicates that the issue/PR needs a Team:* label labels Nov 8, 2021
@elasticmachine
Copy link
Collaborator

elasticmachine commented Nov 8, 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: 2021-11-09T06:33:03.342+0000

  • Duration: 101 min 39 sec

  • Commit: a3ab819

Test stats 🧪

Test Results
Failed 0
Passed 54160
Skipped 5345
Total 59505

💚 Flaky test report

Tests succeeded.

🤖 GitHub comments

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

  • /test : Re-trigger the build.

  • /package : Generate the packages and run the E2E tests.

  • /beats-tester : Run the installation tests with beats-tester.

@mdelapenya mdelapenya merged commit 137bc7b into elastic:master Nov 9, 2021
mergify bot pushed a commit that referenced this pull request Nov 9, 2021
mergify bot pushed a commit that referenced this pull request Nov 9, 2021
mergify bot pushed a commit that referenced this pull request Nov 9, 2021
mdelapenya added a commit that referenced this pull request Nov 9, 2021
(cherry picked from commit 137bc7b)

Co-authored-by: Manuel de la Peña <[email protected]>
mdelapenya added a commit that referenced this pull request Nov 9, 2021
(cherry picked from commit 137bc7b)

Co-authored-by: Manuel de la Peña <[email protected]>
mdelapenya added a commit that referenced this pull request Nov 9, 2021
(cherry picked from commit 137bc7b)

Co-authored-by: Manuel de la Peña <[email protected]>
leweafan pushed a commit to leweafan/beats that referenced this pull request Apr 28, 2023
(cherry picked from commit d29a7e2)

Co-authored-by: Manuel de la Peña <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport-v7.15.0 Automated backport with mergify backport-v7.16.0 Automated backport with mergify backport-v8.0.0 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