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

[SURE-9588] Issue setting folder S3 on etcd backups #499

Closed
aldelsa opened this issue Nov 13, 2024 · 2 comments · Fixed by #549
Closed

[SURE-9588] Issue setting folder S3 on etcd backups #499

aldelsa opened this issue Nov 13, 2024 · 2 comments · Fixed by #549
Assignees
Labels
JIRA must shout priority/important-soon Must be staffed and worked on either currently, or very soon, ideally in time for the next release. triage/accepted Indicates an issue or PR is ready to be actively worked on.
Milestone

Comments

@aldelsa
Copy link

aldelsa commented Nov 13, 2024

Hi,
We are trying to configure the etcd backup using S3 bucket parameters in the RKE2ControlPlane manifest:

  etcd:
      backupConfig:
        retention: "10"
        scheduleCron: "*/3 * * * *"
        s3:
          bucket: backups-rancher
          endpoint: test-rancher.com
          folder: backup
          enforceSslVerify: true
          s3CredentialSecret:
            apiVersion: v1
            kind: Secret
            namespace: rke2test
            name: creds-etcd-backup

The problem is the next part of the code:

rke2ServerConfig.EtcdSnapshotDir = opts.ServerConfig.Etcd.BackupConfig.S3.Folder

If I use that s3 folder, I get "directory does not exist" message from rke2-server service because that folder does not exist on the machine for etcd backup, but it exists in my bucket.

The parameter folder should be independent from the EtcdSnapshotDir.

Thanks

@aldelsa
Copy link
Author

aldelsa commented Nov 13, 2024

I found this PR #278 @furkatgofurov7 but I don't understand the reason of that.

@alexander-demicev alexander-demicev added triage/accepted Indicates an issue or PR is ready to be actively worked on. priority/important-soon Must be staffed and worked on either currently, or very soon, ideally in time for the next release. labels Jan 3, 2025
@kkaempf kkaempf added this to the v0.11.0 milestone Jan 3, 2025
@furkatgofurov7 furkatgofurov7 self-assigned this Jan 8, 2025
@furkatgofurov7
Copy link
Contributor

I found this PR #278 @furkatgofurov7 but I don't understand the reason of that.

@aldelsa thanks for reporting the issue. I opened #549 which reverts the commit (the initial idea was to differentiate the backup location that is later consumed by other project, however it was not a correct assumption to change it the way it is now), PTAL and let me know if it solves your issue.

@kkaempf kkaempf added the JIRA must shout label Jan 10, 2025
@kkaempf kkaempf changed the title Issue setting folder S3 on etcd backups [SURE-9588] Issue setting folder S3 on etcd backups Jan 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
JIRA must shout priority/important-soon Must be staffed and worked on either currently, or very soon, ideally in time for the next release. triage/accepted Indicates an issue or PR is ready to be actively worked on.
Development

Successfully merging a pull request may close this issue.

4 participants