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

chore: remove slack notifications to infra-release-notify #2084

Merged
merged 2 commits into from
Feb 1, 2022

Conversation

jmlrt
Copy link
Member

@jmlrt jmlrt commented Jan 31, 2022

This commit is removing the Slack notifications to
#infra-release-notify channel.

Following elastic/observability-robots#1002,
the ownership of o11y Helm charts should move to o11y team.

cc @elastic/infra-release

This commit is removing the Slack notifications to
`#infra-release-notify` channel.

Following elastic/observability-robots#1002,
the ownership of o11y Helm charts should move to o11y team.
@mergify
Copy link
Contributor

mergify bot commented Jan 31, 2022

This pull request does not have a backport label. Could you fix it @jmlrt? 🙏
To fixup this pull request, you need to add the backport labels for the needed
branches, such as:

  • backport-v./d./d./d is the label to automatically backport to the 7./d branch. /d is the digit
    NOTE: backport-skip has been added to this pull request.

@mergify mergify bot added the backport-skip Skip notification from the automated backport with mergify label Jan 31, 2022
@jmlrt jmlrt added the helm label Jan 31, 2022
@elasticmachine
Copy link
Contributor

elasticmachine commented Jan 31, 2022

💚 Flaky test report

Tests succeeded.

🤖 GitHub comments

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

  • /test : Re-trigger the build.

@jmlrt
Copy link
Member Author

jmlrt commented Feb 1, 2022

👋🏻 @mdelapenya, is there something to do to fix the 2 failing tests, or should we admin-merge? In addition, is there a need to backport to other branches?

@mdelapenya
Copy link
Contributor

mdelapenya commented Feb 1, 2022

The errors are not related to this PR, and they are in upstream (we are on it).

About the backports, we do run the Helm charts tests for main (== 8.1.0-SNAPSHOT), main-1 (8.0.0-SNAPSHOT) and main-2 (7.17 and 7.16), so we still need this backported as this is a multibranch pipeline (tip: we use the mbp suffix everywhere)

@jmlrt jmlrt added backport-v7.16.0 Automated backport with mergify backport-v7.17.0 Automated backport with mergify backport-v8.0.0 Automated backport with mergify and removed backport-skip Skip notification from the automated backport with mergify labels Feb 1, 2022
@jmlrt
Copy link
Member Author

jmlrt commented Feb 1, 2022

Thanks 👍🏻
I've added the backport labels and will let you merge when tests are fixed upstream then.

@mdelapenya
Copy link
Contributor

Thanks 👍🏻 I've added the backport labels and will let you merge when tests are fixed upstream then.

I think we are good to merge even though the upstream tests are broken, as we are in the middle of stabilising them.

Thanks for your help here

@mdelapenya mdelapenya merged commit a52cef9 into elastic:main Feb 1, 2022
mergify bot pushed a commit that referenced this pull request Feb 1, 2022
This commit is removing the Slack notifications to
`#infra-release-notify` channel.

Following elastic/observability-robots#1002,
the ownership of o11y Helm charts should move to o11y team.

(cherry picked from commit a52cef9)
mergify bot pushed a commit that referenced this pull request Feb 1, 2022
This commit is removing the Slack notifications to
`#infra-release-notify` channel.

Following elastic/observability-robots#1002,
the ownership of o11y Helm charts should move to o11y team.

(cherry picked from commit a52cef9)
mergify bot pushed a commit that referenced this pull request Feb 1, 2022
This commit is removing the Slack notifications to
`#infra-release-notify` channel.

Following elastic/observability-robots#1002,
the ownership of o11y Helm charts should move to o11y team.

(cherry picked from commit a52cef9)
@jmlrt jmlrt deleted the remove-release-slack-notifs-main branch February 7, 2022 09:20
jmlrt added a commit that referenced this pull request Feb 7, 2022
)

This commit is removing the Slack notifications to
`#infra-release-notify` channel.

Following elastic/observability-robots#1002,
the ownership of o11y Helm charts should move to o11y team.

(cherry picked from commit a52cef9)

Co-authored-by: Julien Mailleret <[email protected]>
jmlrt added a commit that referenced this pull request Feb 7, 2022
)

This commit is removing the Slack notifications to
`#infra-release-notify` channel.

Following elastic/observability-robots#1002,
the ownership of o11y Helm charts should move to o11y team.

(cherry picked from commit a52cef9)

Co-authored-by: Julien Mailleret <[email protected]>
jmlrt added a commit that referenced this pull request Feb 7, 2022
)

This commit is removing the Slack notifications to
`#infra-release-notify` channel.

Following elastic/observability-robots#1002,
the ownership of o11y Helm charts should move to o11y team.

(cherry picked from commit a52cef9)

Co-authored-by: Julien Mailleret <[email protected]>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
backport-v7.16.0 Automated backport with mergify backport-v7.17.0 Automated backport with mergify backport-v8.0.0 Automated backport with mergify helm
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants