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

fix: increase scheduled build timeouts to 90 min #2294

Merged
merged 2 commits into from
Mar 31, 2022

Conversation

mdelapenya
Copy link
Contributor

What does this PR do?

It increases the build timeout of the scheduled, nightlky builds to 90 min

Why is it important?

Although the pipeline helper times out at 90 min, the trigger build (the nightly) is using 1h, so it dies before expected.

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)

Related issues

@mdelapenya mdelapenya added backport-v7.17.0 Automated backport with mergify backport-v8.0.0 Automated backport with mergify backport-v8.1.0 Automated backport with mergify labels Mar 31, 2022
@mdelapenya mdelapenya self-assigned this Mar 31, 2022
@mdelapenya mdelapenya requested a review from a team March 31, 2022 06:06
@elasticmachine
Copy link
Contributor

elasticmachine commented Mar 31, 2022

💔 Build 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-03-31T08:46:20.255+0000

  • Duration: 53 min 39 sec

Test stats 🧪

Test Results
Failed 0
Passed 309
Skipped 0
Total 309

Steps errors 5

Expand to view the steps failures

Executing ansible-playbook
  • Took 5 min 13 sec . View more details here
  • Description: /var/lib/jenkins/workspace/PR-2294-2-4a5931ff-dc0a-475f-9c62-4a82ae3b02cb/.venv/bin/ansible-playbook --private-key="/var/lib/jenkins/workspace/PR-2294-2-4a5931ff-dc0a-475f-9c62-4a82ae3b02cb/e2essh" --extra-vars="workspace=/var/lib/jenkins/workspace/PR-2294-2-4a5931ff-dc0a-475f-9c62-4a82ae3b02cb/ runId=a682d668 sshPublicKey=/var/lib/jenkins/workspace/PR-2294-2-4a5931ff-dc0a-475f-9c62-4a82ae3b02cb/e2essh.pub" --ssh-common-args='-o StrictHostKeyChecking=no -o UserKnownHostsFile=/dev/null' /var/lib/jenkins/workspace/PR-2294-2-4a5931ff-dc0a-475f-9c62-4a82ae3b02cb/.ci/ansible/playbook.yml -t start-node --extra-vars="buildURL=https://beats-ci.elastic.co/job/e2e-tests/job/e2e-testing-mbp/job/PR-2294/2/ gitSha=0f965ef2f3db7932a70c73d132de09adee1a5d87 stackRunner=18.188.31.133 nodeUser=admin suite=fleet nodeLabel=debian_arm64 nodeImage=ami-06dac44ad759182bd nodeInstanceType=a1.large"
Sleep
  • Took 0 min 5 sec . View more details here
  • Description: 5
Read yaml from files in the workspace or text.
  • Took 0 min 0 sec . View more details here
  • Description: /var/lib/jenkins/workspace/PR-2294-2-4a5931ff-dc0a-475f-9c62-4a82ae3b02cb/debian_arm64-sshhosts
Executing ansible-playbook
  • Took 1 min 2 sec . View more details here
  • Description: /var/lib/jenkins/workspace/PR-2294-2-91fafa77-f1a3-46be-bcc3-817422b63b23/.venv/bin/ansible-playbook --private-key="/var/lib/jenkins/workspace/PR-2294-2-91fafa77-f1a3-46be-bcc3-817422b63b23/e2essh" --extra-vars="workspace=/var/lib/jenkins/workspace/PR-2294-2-91fafa77-f1a3-46be-bcc3-817422b63b23/ runId=e6c0217d sshPublicKey=/var/lib/jenkins/workspace/PR-2294-2-91fafa77-f1a3-46be-bcc3-817422b63b23/e2essh.pub" --ssh-common-args='-o StrictHostKeyChecking=no -o UserKnownHostsFile=/dev/null' /var/lib/jenkins/workspace/PR-2294-2-91fafa77-f1a3-46be-bcc3-817422b63b23/.ci/ansible/playbook.yml -i "18.221.195.201," -t setup-node --extra-vars="buildURL=https://beats-ci.elastic.co/job/e2e-tests/job/e2e-testing-mbp/job/PR-2294/2/ gitSha=0f965ef2f3db7932a70c73d132de09adee1a5d87 stackRunner=18.188.31.133 nodeUser=admin suite=fleet nodeLabel=debian_amd64 nodeImage=ami-0d90bed76900e679a nodeInstanceType=c5.4xlarge"
Sleep
  • Took 0 min 5 sec . View more details here
  • Description: 5

💚 Flaky test report

Tests succeeded.

🤖 GitHub comments

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

  • /test : Re-trigger the build.

The built-in step uses "!isPR()" as fallback if the parameter is not set
@mdelapenya mdelapenya added the Team:Automation Label for the Observability productivity team label Mar 31, 2022
@mdelapenya mdelapenya merged commit ad5fc9f into elastic:main Mar 31, 2022
mergify bot pushed a commit that referenced this pull request Mar 31, 2022
* fix: increase scheduled build timeouts to 90 min

* fix: explicitly avoid slack notifications

The built-in step uses "!isPR()" as fallback if the parameter is not set

(cherry picked from commit ad5fc9f)
mergify bot pushed a commit that referenced this pull request Mar 31, 2022
* fix: increase scheduled build timeouts to 90 min

* fix: explicitly avoid slack notifications

The built-in step uses "!isPR()" as fallback if the parameter is not set

(cherry picked from commit ad5fc9f)
mergify bot pushed a commit that referenced this pull request Mar 31, 2022
* fix: increase scheduled build timeouts to 90 min

* fix: explicitly avoid slack notifications

The built-in step uses "!isPR()" as fallback if the parameter is not set

(cherry picked from commit ad5fc9f)
mdelapenya added a commit that referenced this pull request Apr 1, 2022
* fix: increase scheduled build timeouts to 90 min

* fix: explicitly avoid slack notifications

The built-in step uses "!isPR()" as fallback if the parameter is not set

(cherry picked from commit ad5fc9f)

Co-authored-by: Manuel de la Peña <[email protected]>
mdelapenya added a commit that referenced this pull request Apr 1, 2022
* fix: increase scheduled build timeouts to 90 min

* fix: explicitly avoid slack notifications

The built-in step uses "!isPR()" as fallback if the parameter is not set

(cherry picked from commit ad5fc9f)

Co-authored-by: Manuel de la Peña <[email protected]>
mdelapenya added a commit that referenced this pull request Apr 1, 2022
* fix: increase scheduled build timeouts to 90 min

* fix: explicitly avoid slack notifications

The built-in step uses "!isPR()" as fallback if the parameter is not set

(cherry picked from commit ad5fc9f)

Co-authored-by: Manuel de la Peña <[email protected]>
@mdelapenya mdelapenya deleted the fix-timeouts branch April 1, 2022 08:16
mdelapenya added a commit to mdelapenya/e2e-testing that referenced this pull request Apr 7, 2022
* fix: increase scheduled build timeouts to 90 min

* fix: explicitly avoid slack notifications

The built-in step uses "!isPR()" as fallback if the parameter is not set
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
backport-v7.17.0 Automated backport with mergify backport-v8.0.0 Automated backport with mergify backport-v8.1.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