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.

Sign up for GitHub

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

Jump to bottom

chart, build, salt: bump loki chart to 6.20.0 #4450

Merged

bert-e merged 2 commits into development/129.0 from improvement/bump-loki-6.16.0

Nov 20, 2024

Contributor

eg-ayoub commented Oct 16, 2024

No description provided.

eg-ayoub requested a review from a team as a code owner

October 16, 2024 08:35

Contributor

bert-e commented Oct 16, 2024

Hello eg-ayoub,

My role is to assist you with the merge of this
pull request. Please type @bert-e help to get information
on this process, or consult the user documentation.

Available options

name	description	privileged	authored
`/after_pull_request`	Wait for the given pull request id to be merged before continuing with the current one.
`/bypass_author_approval`	Bypass the pull request author's approval	⭐
`/bypass_build_status`	Bypass the build and test status	⭐
`/bypass_commit_size`	Bypass the check on the size of the changeset `TBA`	⭐
`/bypass_incompatible_branch`	Bypass the check on the source branch prefix	⭐
`/bypass_jira_check`	Bypass the Jira issue check	⭐
`/bypass_peer_approval`	Bypass the pull request peers' approval	⭐
`/bypass_leader_approval`	Bypass the pull request leaders' approval	⭐
`/approve`	Instruct Bert-E that the author has approved the pull request.		✍️
`/create_pull_requests`	Allow the creation of integration pull requests.
`/create_integration_branches`	Allow the creation of integration branches.
`/no_octopus`	Prevent Wall-E from doing any octopus merge and use multiple consecutive merge instead
`/unanimity`	Change review acceptance criteria from `one reviewer at least` to `all reviewers`
`/wait`	Instruct Bert-E not to run until further notice.

Available commands

name	description	privileged
`/help`	Print Bert-E's manual in the pull request.
`/status`	Print Bert-E's current status in the pull request `TBA`
`/clear`	Remove all comments from Bert-E from the history `TBA`
`/retry`	Re-start a fresh build `TBA`
`/build`	Re-start a fresh build `TBA`
`/force_reset`	Delete integration branches & pull requests, and restart merge process from the beginning.
`/reset`	Try to remove integration branches unless there are commits on them which do not appear on the source branch.

Status report is not available.

Contributor

bert-e commented Oct 16, 2024

Waiting for approval

The following approvals are needed before I can proceed with the merge:

the author
one peer

Peer approvals must include at least 1 approval from the following list:

eg-ayoub force-pushed the improvement/bump-loki-6.16.0 branch 3 times, most recently from 1db61a7 to 2f65925 Compare

October 17, 2024 14:26

Contributor

bert-e commented Oct 17, 2024

Waiting for approval

The following approvals are needed before I can proceed with the merge:

the author
one peer

Peer approvals must include at least 1 approval from the following list:

Contributor

bert-e commented Oct 17, 2024

Waiting for approval

The following approvals are needed before I can proceed with the merge:

the author
one peer

Peer approvals must include at least 1 approval from the following list:

eg-ayoub force-pushed the improvement/bump-loki-6.16.0 branch from 2f65925 to d1a10cd Compare

October 18, 2024 07:19

Contributor

bert-e commented Oct 21, 2024

Waiting for approval

The following approvals are needed before I can proceed with the merge:

the author
one peer

Peer approvals must include at least 1 approval from the following list:

eg-ayoub force-pushed the improvement/bump-loki-6.16.0 branch from d1a10cd to 7455870 Compare

October 22, 2024 08:50

Contributor

bert-e commented Oct 22, 2024

Waiting for approval

The following approvals are needed before I can proceed with the merge:

the author
one peer

Peer approvals must include at least 1 approval from the following list:

eg-ayoub force-pushed the improvement/bump-loki-6.16.0 branch from 7455870 to f915305 Compare

October 22, 2024 13:44

Contributor

bert-e commented Oct 22, 2024

Waiting for approval

The following approvals are needed before I can proceed with the merge:

the author
one peer

Peer approvals must include at least 1 approval from the following list:

eg-ayoub force-pushed the improvement/bump-loki-6.16.0 branch 2 times, most recently from c86a50a to eb0977d Compare

October 23, 2024 14:16

eg-ayoub force-pushed the improvement/bump-loki-6.16.0 branch from eb0977d to 1cbf9ee Compare

October 31, 2024 13:44

Contributor

bert-e commented Oct 31, 2024

Waiting for approval

The following approvals are needed before I can proceed with the merge:

the author
one peer

Peer approvals must include at least 1 approval from the following list:

eg-ayoub commented

View reviewed changes

charts/loki.yaml Show resolved Hide resolved

eg-ayoub changed the title ~~chart, build, salt: bump loki chart to 6.16.0~~ chart, build, salt: bump loki chart to 6.18.0

Contributor

bert-e commented Nov 7, 2024

Waiting for approval

The following approvals are needed before I can proceed with the merge:

the author
one peer

Peer approvals must include at least 1 approval from the following list:

eg-ayoub commented

View reviewed changes

salt/metalk8s/addons/logging/loki/config/loki.yaml Outdated Show resolved Hide resolved

eg-ayoub commented

View reviewed changes

salt/metalk8s/addons/logging/loki/config/loki.yaml Outdated Show resolved Hide resolved

eg-ayoub commented

View reviewed changes

salt/metalk8s/addons/logging/loki/config/loki.yaml Outdated Show resolved Hide resolved

eg-ayoub commented

View reviewed changes

salt/metalk8s/addons/logging/loki/config/loki.yaml Outdated Show resolved Hide resolved

eg-ayoub force-pushed the improvement/bump-loki-6.16.0 branch from 1cbf9ee to ef6cfe3 Compare

November 8, 2024 08:28

Contributor

bert-e commented Nov 8, 2024

Waiting for approval

The following approvals are needed before I can proceed with the merge:

the author
one peer

Peer approvals must include at least 1 approval from the following list:

eg-ayoub force-pushed the improvement/bump-loki-6.16.0 branch from ef6cfe3 to 44f0569 Compare

November 19, 2024 08:42

Contributor

bert-e commented Nov 19, 2024

Waiting for approval

The following approvals are needed before I can proceed with the merge:

the author
one peer

Peer approvals must include at least 1 approval from the following list:

eg-ayoub force-pushed the improvement/bump-loki-6.16.0 branch from 44f0569 to 6962bc4 Compare

November 19, 2024 08:47

eg-ayoub changed the title ~~chart, build, salt: bump loki chart to 6.18.0~~ chart, build, salt: bump loki chart to 6.19.0

Contributor

bert-e commented Nov 19, 2024

Waiting for approval

The following approvals are needed before I can proceed with the merge:

the author
one peer

Peer approvals must include at least 1 approval from the following list:

eg-ayoub force-pushed the improvement/bump-loki-6.16.0 branch from e940885 to 1f59e3c Compare

November 20, 2024 08:32

Contributor

bert-e commented Nov 20, 2024

Waiting for approval

The following approvals are needed before I can proceed with the merge:

the author
one peer

Peer approvals must include at least 1 approval from the following list:

eg-ayoub changed the title ~~chart, build, salt: bump loki chart to 6.19.0~~ chart, build, salt: bump loki chart to 6.20.0

eg-ayoub added 2 commits

November 20, 2024 09:34


          chart, build, salt: bump loki chart to 6.20.0

d129a29

apply breaking changes to loki config.


          fix e2e test for loki

0a9b2c9

eg-ayoub force-pushed the improvement/bump-loki-6.16.0 branch from 1f59e3c to 0a9b2c9 Compare

November 20, 2024 08:34

m4nch0t approved these changes

View reviewed changes

Contributor Author

eg-ayoub commented Nov 20, 2024

/approve

Contributor

bert-e commented Nov 20, 2024

In the queue

The changeset has received all authorizations and has been added to the
relevant queue(s). The queue(s) will be merged in the target development
branch(es) as soon as builds have passed.

The changeset will be merged in:

✔️ development/129.0

The following branches will NOT be impacted:

development/123.0
development/124.0
development/124.1
development/125.0
development/126.0
development/127.0
development/128.0
development/2.0
development/2.1
development/2.10
development/2.11
development/2.2
development/2.3
development/2.4
development/2.5
development/2.6
development/2.7
development/2.8
development/2.9

There is no action required on your side. You will be notified here once
the changeset has been merged. In the unlikely event that the changeset
fails permanently on the queue, a member of the admin team will
contact you to help resolve the matter.

IMPORTANT

Please do not attempt to modify this pull request.

Any commit you add on the source branch will trigger a new cycle after the
current queue is merged.
Any commit you add on one of the integration branches will be lost.

If you need this pull request to be removed from the queue, please contact a
member of the admin team now.

The following options are set: approve

Contributor

bert-e commented Nov 20, 2024

I have successfully merged the changeset of this pull request
into targetted development branches:

✔️ development/129.0

The following branches have NOT changed:

development/123.0
development/124.0
development/124.1
development/125.0
development/126.0
development/127.0
development/128.0
development/2.0
development/2.1
development/2.10
development/2.11
development/2.2
development/2.3
development/2.4
development/2.5
development/2.6
development/2.7
development/2.8
development/2.9

Please check the status of the associated issue None.

Goodbye eg-ayoub.

bert-e merged commit f7fc60a into development/129.0

27 checks passed

bert-e deleted the improvement/bump-loki-6.16.0 branch

November 20, 2024 15:05

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet