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

Add service account to backup job #154

Closed
wants to merge 1 commit into from
Closed

Conversation

pichouk
Copy link

@pichouk pichouk commented Jan 9, 2025

I suggest to add the Kubernetes Service Account to the backup job, allowing to grant access to external object storage like S3 or GCS using IRSA or Workload Identity for example.

Let me know if you prefer this to be optional.

@mihailradkov
Copy link
Contributor

Hey @pichouk

Could you provide more details on how the service account would be used in the backup CronJob?

Note that the actual backup upload is handled by GraphDB, and the chart support providing a ServiceAccount for the StatefulSet specifically for IRSA (there is an example here). The CronJob just triggers the backup process in GraphDB via REST so I'm not sure how a ServiceAccount in the CronJob would change the overall flow.

@pichouk
Copy link
Author

pichouk commented Jan 14, 2025

Shame on me. I though the backup was uploaded by the job and it seems I did not even try without this setup. It just works as expected out of the box... Sorry for the noise, I feel a bit stupid right now 😬

@pichouk pichouk closed this Jan 14, 2025
@pichouk pichouk deleted the patch-1 branch January 16, 2025 07:25
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.

2 participants