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

[synthetics-private-location] build: 👷 Bump PL to 1.54.0 #1633

Closed

Conversation

GuilhermeBorges
Copy link

What this PR does / why we need it:

The Synthetics Private Location was updated to version 1.54.0.
This PR bumps the version in the associated Chart.

Checklist

[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

  • Chart Version bumped
  • Documentation has been updated with helm-docs (run: .github/helm-docs.sh)
  • CHANGELOG.md has been updated
  • Variables are documented in the README.md
  • For Datadog Operator chart or value changes update the test baselines (run: make update-test-baselines)

@GuilhermeBorges GuilhermeBorges requested a review from a team as a code owner December 12, 2024 15:52
@github-actions github-actions bot added the chart/synthetics-private-location This issue or pull request is related to the synthetics-private-location chart label Dec 12, 2024
@GuilhermeBorges
Copy link
Author

/merge

@dd-devflow
Copy link

dd-devflow bot commented Dec 16, 2024

Devflow running: /merge

View all feedbacks in Devflow UI.


2024-12-16 12:38:02 UTC ℹ️ MergeQueue: waiting for PR to be ready

This merge request is not mergeable yet, because of pending checks/missing approvals. It will be added to the queue as soon as checks pass and/or get approvals.
Note: if you pushed new commits since the last approval, you may need additional approval.
You can remove it from the waiting list with /remove command.


2024-12-16 16:38:13 UTC ⚠️ MergeQueue: This merge request was unqueued

This merge request was unqueued

@alexlo03
Copy link

@GuilhermeBorges can you hit it again? thanks

@GuilhermeBorges
Copy link
Author

/merge

@dd-devflow
Copy link

dd-devflow bot commented Dec 17, 2024

Devflow running: /merge

View all feedbacks in Devflow UI.


2024-12-17 10:15:09 UTC ℹ️ MergeQueue: pull request added to the queue

The median merge time in main is 37m.


2024-12-17 10:17:47 UTC 🚨 MergeQueue: This merge request is in error

Gitlab pipeline didn't start on its own and we were unable to create it... Please retry.

@alexlo03
Copy link

@GuilhermeBorges can you hit it again? thanks

@GuilhermeBorges
Copy link
Author

/merge

@dd-devflow
Copy link

dd-devflow bot commented Dec 17, 2024

Devflow running: /merge

View all feedbacks in Devflow UI.


2024-12-17 16:40:46 UTC ℹ️ MergeQueue: pull request added to the queue

The median merge time in main is 37m.


2024-12-17 16:44:12 UTC 🚨 MergeQueue: This merge request is in error

Gitlab pipeline didn't start on its own and we were unable to create it... Please retry.

@GuilhermeBorges
Copy link
Author

@alexlo03 do you know what went wrong?

@alexlo03
Copy link

@GuilhermeBorges computers were a mistake

But more relevant to this problem: I cannot see what is happening in your DevFlow thing - https://devflow.static-app.us1.prod.dog/logs/0f56e7f1-6558-4c42-ae54-52be726ff32a_36

@GuilhermeBorges
Copy link
Author

@alexlo03 I’ll give this final “retry” a shot, and if it doesn’t resolve the issue. Who would be the best person to reach out to for a better understanding of what might be going wrong?

@GuilhermeBorges
Copy link
Author

/merge

@dd-devflow
Copy link

dd-devflow bot commented Dec 19, 2024

Devflow running: /merge

View all feedbacks in Devflow UI.


2024-12-19 09:56:58 UTC ℹ️ MergeQueue: pull request added to the queue

The median merge time in main is 37m.


2024-12-19 09:59:35 UTC 🚨 MergeQueue: This merge request is in error

Gitlab pipeline didn't start on its own and we were unable to create it... Please retry.

@GuilhermeBorges GuilhermeBorges requested a review from a team as a code owner December 19, 2024 14:06
@GuilhermeBorges
Copy link
Author

/merge

@dd-devflow
Copy link

dd-devflow bot commented Dec 19, 2024

Devflow running: /merge

View all feedbacks in Devflow UI.


2024-12-19 14:18:48 UTC ℹ️ MergeQueue: pull request added to the queue

The median merge time in main is 37m.


2024-12-19 14:20:37 UTC 🚨 MergeQueue: This merge request is in error

mergequeue build completed successfully, but the github api returned an error while merging the pr.
It's probably because:

  • target branch of PR is restricted to only allow up-to-date branches, but the pr is now outdated
Details

Error: PUT https://api.github.com/repos/DataDog/helm-charts/pulls/1633/merge: 405 Waiting on code owner review from DataDog/container-helm-chart-maintainers. Required status check "pr-validated" is expected. []

FullStacktrace:
activity error (type: github.GithubService_MergePullRequest, scheduledEventID: 41, startedEventID: 42, identity: 1@github-worker-689795c6fc-6xwxg@): PUT https://api.github.com/repos/DataDog/helm-charts/pulls/1633/merge: 405 Waiting on code owner review from DataDog/container-helm-chart-maintainers. Required status check "pr-validated" is expected. [] (type: GitFailure, retryable: false): PUT https://api.github.com/repos/DataDog/helm-charts/pulls/1633/merge: 405 Waiting on code owner review from DataDog/container-helm-chart-maintainers. Required status check "pr-validated" is expected. [] (type: ErrorResponse, retryable: true)

@GuilhermeBorges GuilhermeBorges force-pushed the guilherme.oliveira/SYNTH-17002/bump-pl-1-54-0 branch from f0d4f2c to 6bf1f7b Compare December 19, 2024 15:09
auto-merge was automatically disabled December 19, 2024 16:36

Pull request was closed

@GuilhermeBorges
Copy link
Author

Declined since we did it through this one: #1646

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chart/synthetics-private-location This issue or pull request is related to the synthetics-private-location chart mergequeue-status: error tools/ci
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants