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

ci(action): auto-bump daily and change slack notification channel #3886

Merged
merged 5 commits into from
Jan 31, 2024

Conversation

v1v
Copy link
Member

@v1v v1v commented Dec 8, 2023

What does this PR do?

Change the slack channel
Run daily

Why is it important?

There will be only 24 hours daily between the golang-crossbuild auto-bumps and this one.

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/fragments using the changelog tool
  • I have added an integration test or an E2E test

Author's Checklist

  • [ ]

How to test this PR locally

Related issues

Similar to elastic/golang-crossbuild#367

Use cases

Screenshots

Logs

Questions to ask yourself

  • How are we going to support this in production?
  • How are we going to measure its adoption?
  • How are we going to debug this?
  • What are the metrics I should take care of?
  • ...

@v1v v1v requested a review from cmacknz December 8, 2023 09:23
@v1v v1v requested a review from a team as a code owner December 8, 2023 09:23
@v1v v1v requested a review from pchila December 8, 2023 09:23
@mergify mergify bot assigned v1v Dec 8, 2023
Copy link
Contributor

mergify bot commented Dec 8, 2023

This pull request does not have a backport label. Could you fix it @v1v? 🙏
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 8./d branch. /d is the digit

NOTE: backport-skip has been added to this pull request.

@v1v
Copy link
Member Author

v1v commented Dec 11, 2023

CI error unrelated to this change, it failed when processing the test results

image

@pierrehilbert
Copy link
Contributor

I fixed the processing step

@v1v
Copy link
Member Author

v1v commented Dec 12, 2023

The CI failures are unrelated to the nature of this PR.

image

@pierrehilbert
Copy link
Contributor

#3901 will fix the test error

Copy link

Quality Gate passed Quality Gate passed

Kudos, no new issues were introduced!

0 New issues
0 Security Hotspots
No Coverage information No data about Coverage
No Duplication information No data about Duplication

See analysis details on SonarQube

@v1v
Copy link
Member Author

v1v commented Dec 19, 2023

Unrelated CI failures

image

@v1v v1v enabled auto-merge (squash) January 30, 2024 09:55
@v1v
Copy link
Member Author

v1v commented Jan 30, 2024

I have no clue what's the reason for the failure in the CI, but this PR does not change anything that the CI can test/verify. It changes the automation in GitHub actions.

@pierrehilbert, how should we proceed?

@pierrehilbert
Copy link
Contributor

This error is a new one, I'm looking at it to report it and if this can't be related to the change, will force merge this one.

@pierrehilbert
Copy link
Contributor

/test

Copy link

Quality Gate passed Quality Gate passed

Kudos, no new issues were introduced!

0 New issues
0 Security Hotspots
No Coverage information No data about Coverage
No Duplication information No data about Duplication

See analysis details on SonarQube

@v1v v1v merged commit d8b446f into main Jan 31, 2024
9 checks passed
@v1v v1v deleted the v1v-patch-1 branch January 31, 2024 17:42
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.

3 participants