Add test for archive object replacement #2170

francoisferrand · 2024-11-17T11:42:27Z

Add test for archive object replacement
Bump cloudserver

This is needed to avoid conflict with internal prometheus service, since we are running in the same cluster. Issue: ZENKO-4876

ServiceMonitor were not scrapped. Issue: ZENKO-4876

When an SHA1 is used which is not the tip of the branch, it is not available after a `git clone` (which fetches only commits reachable from current branches). This causes issues if a build is started while the component branch with the commit has been rewritten (rebase...) To overcome this, the safe way is to explicitely fetch the commit: which works for any commit (branch, tag, sha1). Issue: ZENKO-4876

Issue: ZENKO-4876

No reason to keep separate, and it helps avoid a github bug/limitation when creating a check: and the Issue: ZENKO-4876

The trigger seem to be aws/aws-sdk-js#1678 ... causing the callback to be called twice, which is a problem in these loops. Issue: ZENKO-4925

Issue: ZENKO-4925

Issue: ZENKO-4926

bert-e · 2024-11-17T11:42:31Z

Hello francoisferrand,

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.

.github/scripts/end2end/configs/zenkoversion.yaml

tests/ctst/features/dmf.feature

tests/ctst/steps/utils/utils.ts

williamlardier

Approving because the comments are not blocking & I know they will be handled before merging

tests/ctst/steps/utils/utils.ts

bert-e · 2024-11-18T15:14:09Z

Request integration branches

Waiting for integration branch creation to be requested by the user.

To request integration branches, please comment on this pull request with the following command:

/create_integration_branches

Alternatively, the /approve and /create_pull_requests commands will automatically
create the integration branches.

Issue: ZENKO-4919

…ent/ZENKO-4919

…rovement/ZENKO-4919

We need to retry `kubectl apply`, esp as mutating web hooks may not be ready, c.f. [1]. [1] cert-manager/cert-manager#2908 Issue: ZENKO-4919

Issue: ZENKO-4919

Add replacement with MPU or copyObject. Issue: ZENKO-4928

Issue: ZENKO-4928

francoisferrand and others added 11 commits November 12, 2024 16:24

Add artesca-root-ca-issuer cluster issuer

9678f8c

Issue: ZENKO-4876

Test that federated metrics are retrieved

7476936

Issue: ZENKO-4876

Set federated prometheus name

8970422

This is needed to avoid conflict with internal prometheus service, since we are running in the same cluster. Issue: ZENKO-4876

Fix prometheus config

9826e4c

ServiceMonitor were not scrapped. Issue: ZENKO-4876

Move kafka dump to folder

33171e8

Issue: ZENKO-4876

Merge alerts workflow

a4cb9ae

No reason to keep separate, and it helps avoid a github bug/limitation when creating a check: and the Issue: ZENKO-4876

Ensure callback is called once in doWhilst with S3 operations

b88d962

The trigger seem to be aws/aws-sdk-js#1678 ... causing the callback to be called twice, which is a problem in these loops. Issue: ZENKO-4925

Improve variable in doWhilst

88384bc

Issue: ZENKO-4925

Bump backbeat 8.6.51

47efee6

Issue: ZENKO-4926

Bump zenko-operator 1.6.4

4a877c7

Issue: ZENKO-4926

williamlardier reviewed Nov 18, 2024

View reviewed changes

.github/scripts/end2end/configs/zenkoversion.yaml Outdated Show resolved Hide resolved

tests/ctst/features/dmf.feature Outdated Show resolved Hide resolved

tests/ctst/steps/utils/utils.ts Show resolved Hide resolved

williamlardier approved these changes Nov 18, 2024

View reviewed changes

francoisferrand force-pushed the improvement/ZENKO-4928 branch from 31a40f4 to a541a97 Compare November 18, 2024 13:01

KillianG reviewed Nov 18, 2024

View reviewed changes

tests/ctst/steps/utils/utils.ts Show resolved Hide resolved

francoisferrand force-pushed the improvement/ZENKO-4928 branch from a541a97 to 1fe721a Compare November 18, 2024 15:13

francoisferrand changed the base branch from development/2.10 to development/2.7 November 18, 2024 15:14

scality deleted a comment from bert-e Nov 18, 2024

francoisferrand force-pushed the improvement/ZENKO-4928 branch 2 times, most recently from 85dabf0 to 5ca1dac Compare November 18, 2024 15:23

francoisferrand added 5 commits November 18, 2024 16:35

Bump zenko-operator 1.6.5

d15032f

Issue: ZENKO-4919

Bump cloudserver 8.8.36

0448960

Issue: ZENKO-4919

Bump sorbet 1.1.13

d202d0b

Issue: ZENKO-4919

Merge remote-tracking branch 'origin/bugfix/ZENKO-4925' into improvem…

62cc574

…ent/ZENKO-4919

Merge remote-tracking branch 'origin/improvement/ZENKO-4876' into imp…

3106930

…rovement/ZENKO-4919

KillianG approved these changes Nov 18, 2024

View reviewed changes

francoisferrand force-pushed the improvement/ZENKO-4928 branch from 5ca1dac to d19a7a9 Compare November 18, 2024 18:05

Fix flaky cluster issuer setup

4679d66

We need to retry `kubectl apply`, esp as mutating web hooks may not be ready, c.f. [1]. [1] cert-manager/cert-manager#2908 Issue: ZENKO-4919

francoisferrand added 5 commits November 18, 2024 19:36

Release Zenko 2.6.70

6cd4998

Issue: ZENKO-4919

Merge branch 'improvement/ZENKO-4919' into w/2.7/improvement/ZENKO-4919

98e2c4f

Release Zenko 2.7.66

29f7c32

Issue: ZENKO-4919

Add test for archive object replacement

47e6917

Add replacement with MPU or copyObject. Issue: ZENKO-4928

ctst: check result of object creation steps

dfa1a65

Issue: ZENKO-4928

francoisferrand force-pushed the improvement/ZENKO-4928 branch from d19a7a9 to dfa1a65 Compare November 18, 2024 20:02

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Add test for archive object replacement #2170

Add test for archive object replacement #2170

francoisferrand commented Nov 17, 2024

bert-e commented Nov 17, 2024

williamlardier left a comment

bert-e commented Nov 18, 2024

Add test for archive object replacement #2170

Are you sure you want to change the base?

Add test for archive object replacement #2170

Conversation

francoisferrand commented Nov 17, 2024

bert-e commented Nov 17, 2024

Hello francoisferrand,

williamlardier left a comment

Choose a reason for hiding this comment

bert-e commented Nov 18, 2024

Request integration branches