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

chore: do not notify build green for fleet nightly builds#2277

Merged
mdelapenya merged 1 commit intoelastic:mainfrom
mdelapenya:slack-notify-fleet-red-builds
Mar 28, 2022
Merged

chore: do not notify build green for fleet nightly builds#2277
mdelapenya merged 1 commit intoelastic:mainfrom
mdelapenya:slack-notify-fleet-red-builds

Conversation

@mdelapenya
Copy link
Copy Markdown
Contributor

What does this PR do?

It removes the Slack notification for fleet's nightly builds.

Why is it important?

The team has shared their concerns about receiving 4 green notifications a day (one per maintained branch) for the nightly builds. They prefer receive failures, only.

I like celebrating success too!

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)

@mdelapenya mdelapenya added Team:Automation Label for the Observability productivity team area:ci Anything related to the CI 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 28, 2022
@mdelapenya mdelapenya self-assigned this Mar 28, 2022
@mdelapenya mdelapenya requested review from a team, jlind23, ph and ruflin March 28, 2022 14:57
@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: 2022-03-28T14:57:36.191+0000

  • Duration: 65 min 38 sec

Test stats 🧪

Test Results
Failed 0
Passed 276
Skipped 0
Total 276

💚 Flaky test report

Tests succeeded.

🤖 GitHub comments

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

  • /test : Re-trigger the build.

@mdelapenya mdelapenya merged commit b211445 into elastic:main Mar 28, 2022
mergify bot pushed a commit that referenced this pull request Mar 28, 2022
mergify bot pushed a commit that referenced this pull request Mar 28, 2022
mergify bot pushed a commit that referenced this pull request Mar 28, 2022
mdelapenya added a commit that referenced this pull request Mar 28, 2022
)

(cherry picked from commit b211445)

Co-authored-by: Manuel de la Peña <mdelapenya@gmail.com>
mdelapenya added a commit that referenced this pull request Mar 28, 2022
)

(cherry picked from commit b211445)

Co-authored-by: Manuel de la Peña <mdelapenya@gmail.com>
mdelapenya added a commit that referenced this pull request Mar 28, 2022
)

(cherry picked from commit b211445)

Co-authored-by: Manuel de la Peña <mdelapenya@gmail.com>
mdelapenya added a commit to mdelapenya/e2e-testing that referenced this pull request Mar 30, 2022
* main: (72 commits)
  fix: always quote variables in shell scripts (elastic#2284)
  chore: bring back system integration scenarios (elastic#2233)
  chore: do not notify build green for fleet nightly builds (elastic#2277)
  chore: include suite, platform and tags in test reports name (elastic#2273)
  bump stack version 8.2.0-dcff22d7 (elastic#2270)
  bump stack version 8.2.0-4509f321 (elastic#2265)
  chore: bump Godog to v0.12.4 (elastic#2245)
  bump stack version 8.2.0-5cc993c1 (elastic#2254)
  bump stack version 8.2.0-82b803a1 (elastic#2250)
  fix: never process elasticsearch CI artifacts (elastic#2246)
  bump stack version 8.2.0-ff67d7b8 (elastic#2242)
  ci: periodic builds for the last two minor versions (elastic#2241)
  bump stack version 8.2.0-9bac538c (elastic#2240)
  fix: add end-of-file line in templates (elastic#2237)
  chore: add Christos to SSH users
  bump stack version 8.2.0-fee3b8d2 (elastic#2234)
  Separate ES workload from Agent spec and make it req for the scenarios (elastic#2203)
  bump stack version 8.2.0-d02c907a (elastic#2228)
  bump stack version 8.2.0-63265ec9 (elastic#2225)
  fix: wrong link (elastic#2220)
  ...
@mdelapenya mdelapenya deleted the slack-notify-fleet-red-builds branch July 13, 2022 09:30
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

area:ci Anything related to the CI 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.

5 participants