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

Merge release-2.52 back to main #14016

Merged
merged 11 commits into from
May 2, 2024
Merged

Merge release-2.52 back to main #14016

merged 11 commits into from
May 2, 2024

Conversation

ArthurSens
Copy link
Member

No description provided.

ArthurSens and others added 11 commits April 24, 2024 09:37
promtool: Fix panic on extended tsdb analyze (#13976)
Signed-off-by: Arthur Silva Sens <arthur.sens@coralogix.com>
Since we already use require.JSONEq in similar cases.

Signed-off-by: Bryan Boreham <bjboreham@gmail.com>
So that tests use the same encoding as the api.

Signed-off-by: Bryan Boreham <bjboreham@gmail.com>
When we want to check just the json encoding.

Signed-off-by: Bryan Boreham <bjboreham@gmail.com>
Signed-off-by: Bryan Boreham <bjboreham@gmail.com>
The typed versions are used when we call from one marshaller to another.

Signed-off-by: Bryan Boreham <bjboreham@gmail.com>
Signed-off-by: Bryan Boreham <bjboreham@gmail.com>
If the underlying data is `nil` the default encoding
will render `"null"` which is not accepted by
(some) Prometheus client libraries.

Signed-off-by: Bryan Boreham <bjboreham@gmail.com>
bugfix: API: encode empty Vector/Matrix as [] not null
@ArthurSens
Copy link
Member Author

Hmmm, I wasn't expecting conflicts in the changelog file...

Not sure if I should be adding extra commits to the release branch to fix this 🤔

@Nexucis
Copy link
Member

Nexucis commented May 2, 2024

@ArthurSens you should create a branch from the release one, resolve the conflict and then merge your branch into the main

@ArthurSens
Copy link
Member Author

@ArthurSens you should create a branch from the release one, resolve the conflict and then merge your branch into the main

I'm not sure how that would work... If I create another branch from the release branch, and then rebase on top of main to solve the conflicts, the PR will contain a lot of extra commits that are already in the main branch 🤔

@Nexucis
Copy link
Member

Nexucis commented May 2, 2024

  1. you create a branch based on the branch "release-2.52"
  2. You merge (not rebase) the main branch into yours.
  3. You solve the conflict
  4. You open a PR with your branch to main
  5. You merge (not squash) the PR.

Is it clearer @ArthurSens ?

@Nexucis Nexucis merged commit 650f4d7 into main May 2, 2024
63 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants