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

enhancement: Added oportuniy set ImagePullSecret #923

Open
wants to merge 80 commits into
base: master
Choose a base branch
from

Conversation

eunames
Copy link

@eunames eunames commented Feb 14, 2024

Summary

Added oportuniy set ImagePullSecret param to manifest of job. It allows use a private registry for images when we use $BACKUP_IMAGE.

Fixes #567

Checklist

For Code changes

  • Categorize the PR by setting a good title and adding one of the labels:
    bug, enhancement, documentation, change, breaking, dependency
    as they show up in the changelog
  • PR contains the label area:operator
  • Link this PR to related issues
  • I have not made any changes in the charts/ directory.

For Helm Chart changes

  • Categorize the PR by setting a good title and adding one of the labels:
    bug, enhancement, documentation, change, breaking, dependency
    as they show up in the changelog
  • PR contains the label area:chart
  • PR contains the chart label, e.g. chart:k8up
  • Variables are documented in the values.yaml using the format required by Helm-Docs.
  • Chart Version bumped if immediate release after merging is planned
  • I have run make chart-docs
  • Link this PR to related code release or other issues.

@eunames eunames requested a review from a team as a code owner February 14, 2024 19:33
@eunames eunames requested review from tobru and Kidswiss and removed request for a team February 14, 2024 19:33
@eunames eunames force-pushed the enhancement/add_imagePullSecret branch from 8f469c9 to ff85fd1 Compare February 14, 2024 19:37
@eunames eunames changed the title Added oportuniy set ImagePullSecret enhancement: Added oportuniy set ImagePullSecret Feb 14, 2024
@@ -6,7 +6,7 @@ keywords:
- backup
- operator
- restic
version: 4.4.3
version: 4.4.4
Copy link
Contributor

@Kidswiss Kidswiss Mar 11, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would actually consider this a feature, can you bump it to 4.5.0 please?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done

namesoname and others added 17 commits April 24, 2024 10:21
Signed-off-by: Name Soname <bx-500@yandex.ru>
Signed-off-by: eunames <eu505@hotmail.com>
Signed-off-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Signed-off-by: eunames <eu505@hotmail.com>
There's a target that is commented-out in `envtest/integration.mk` that
still shows up in the output of `make help`.

This commit filters out lines that start with a comment (`#`)

Signed-off-by: Manuel Hutter <manuel@hutter.io>
Signed-off-by: eunames <eu505@hotmail.com>
¯\_(ツ)_/¯

Signed-off-by: Manuel Hutter <manuel@hutter.io>
Signed-off-by: eunames <eu505@hotmail.com>
Signed-off-by: Pooya Azarpour <pooya_azarpour@yahoo.com>
Signed-off-by: eunames <eu505@hotmail.com>
Signed-off-by: Pooya Azarpour <pooya_azarpour@yahoo.com>
Signed-off-by: eunames <eu505@hotmail.com>
…tation). Ref: k8up-io#803

Signed-off-by: Pooya Azarpour <pooya_azarpour@yahoo.com>
Signed-off-by: eunames <eu505@hotmail.com>
…reSingleFile`

Signed-off-by: Pooya Azarpour <pooya_azarpour@yahoo.com>
Signed-off-by: eunames <eu505@hotmail.com>
Signed-off-by: Pooya Azarpour <pooya_azarpour@yahoo.com>
Signed-off-by: eunames <eu505@hotmail.com>
…kind, the pod don't have OwnerReferences)

Signed-off-by: Pooya Azarpour <pooya_azarpour@yahoo.com>
Signed-off-by: eunames <eu505@hotmail.com>
[FIX] Fix helm chart of minio (The address of helm.min.io changed to charts.min.io) and values of chart has been change

Signed-off-by: Pooya Azarpour <pooya_azarpour@yahoo.com>
Signed-off-by: eunames <eu505@hotmail.com>
Signed-off-by: Pooya Azarpour <pooya_azarpour@yahoo.com>
Signed-off-by: eunames <eu505@hotmail.com>
Signed-off-by: Benjamin Ritter <Benjamin.Ritter@protonmail.com>
Signed-off-by: eunames <eu505@hotmail.com>
Signed-off-by: Benjamin Ritter <Benjamin.Ritter@protonmail.com>
Signed-off-by: eunames <eu505@hotmail.com>
Signed-off-by: Tobias Brunner <tobias.brunner@vshn.ch>
Signed-off-by: eunames <eu505@hotmail.com>
Signed-off-by: Tobias Brunner <tobias.brunner@vshn.ch>
Signed-off-by: eunames <eu505@hotmail.com>
Signed-off-by: Tobias Brunner <tobias.brunner@vshn.ch>
Signed-off-by: eunames <eu505@hotmail.com>
poyaz and others added 28 commits April 24, 2024 10:21
This test contains below sections:
- Testing backup API for TLS and mTLS mode
- Testing restore API in pvc for TLS and mTLS mode
- Testing restore API in S3 for TLS and mTLS mode
- Testing archive API in S3 for TLS and mTLS mode
- Testin check API for TLS and mTLS mode

Signed-off-by: poyaz <pooya_azarpour@yahoo.com>
Signed-off-by: eunames <eu505@hotmail.com>
Signed-off-by: poyaz <pooya_azarpour@yahoo.com>
Signed-off-by: eunames <eu505@hotmail.com>
…NT_KEY_FILE env instead of filling TLS and mTls options in restore method

Signed-off-by: poyaz <pooya_azarpour@yahoo.com>
Signed-off-by: eunames <eu505@hotmail.com>
…is added

Signed-off-by: poyaz <pooya_azarpour@yahoo.com>
Signed-off-by: eunames <eu505@hotmail.com>
Signed-off-by: poyaz <pooya_azarpour@yahoo.com>
Signed-off-by: eunames <eu505@hotmail.com>
Signed-off-by: poyaz <pooya_azarpour@yahoo.com>
Signed-off-by: eunames <eu505@hotmail.com>
…unt, and options

Signed-off-by: poyaz <pooya_azarpour@yahoo.com>
Signed-off-by: eunames <eu505@hotmail.com>
Signed-off-by: poyaz <pooya_azarpour@yahoo.com>
Signed-off-by: eunames <eu505@hotmail.com>
Signed-off-by: poyaz <pooya_azarpour@yahoo.com>
Signed-off-by: eunames <eu505@hotmail.com>
Also fixing bug in get lentgh of archive object in minio-mc

Signed-off-by: poyaz <pooya_azarpour@yahoo.com>
Signed-off-by: eunames <eu505@hotmail.com>
Signed-off-by: poyaz <pooya_azarpour@yahoo.com>
Signed-off-by: eunames <eu505@hotmail.com>
…s of Golang

Signed-off-by: poyaz <pooya_azarpour@yahoo.com>
Signed-off-by: eunames <eu505@hotmail.com>
Signed-off-by: poyaz <pooya_azarpour@yahoo.com>
Signed-off-by: eunames <eu505@hotmail.com>
Signed-off-by: Pooya Azarpour <pooya_azarpour@yahoo.com>
…formatter)

Signed-off-by: Pooya Azarpour <pooya_azarpour@yahoo.com>
Signed-off-by: eunames <eu505@hotmail.com>
Signed-off-by: Pooya Azarpour <pooya_azarpour@yahoo.com>
Signed-off-by: eunames <eu505@hotmail.com>
Signed-off-by: Pooya Azarpour <pooya_azarpour@yahoo.com>
Signed-off-by: eunames <eu505@hotmail.com>
Signed-off-by: Pooya Azarpour <pooya_azarpour@yahoo.com>
Signed-off-by: eunames <eu505@hotmail.com>
Signed-off-by: poyaz <pooya_azarpour@yahoo.com>
Signed-off-by: eunames <eu505@hotmail.com>
These functions is created in utils:
- AppendTLSOptionsArgs: for generate env for backend and restore specs
- AttachTLSVolumes: for create volumes for pods
AttachTLSVolumeMounts: for create volumeMount for backend and restore specs

Signed-off-by: poyaz <pooya_azarpour@yahoo.com>
Signed-off-by: eunames <eu505@hotmail.com>
Signed-off-by: poyaz <pooya_azarpour@yahoo.com>
Signed-off-by: eunames <eu505@hotmail.com>
Signed-off-by: poyaz <pooya_azarpour@yahoo.com>
Signed-off-by: eunames <eu505@hotmail.com>
… case and spend too much time

Move restore and archive test case to two separated files

Signed-off-by: poyaz <pooya_azarpour@yahoo.com>
Signed-off-by: eunames <eu505@hotmail.com>
Signed-off-by: poyaz <pooya_azarpour@yahoo.com>
Signed-off-by: eunames <eu505@hotmail.com>
Signed-off-by: poyaz <pooya_azarpour@yahoo.com>
Signed-off-by: eunames <eu505@hotmail.com>
Signed-off-by: poyaz <pooya_azarpour@yahoo.com>
Signed-off-by: eunames <eu505@hotmail.com>
Signed-off-by: poyaz <pooya_azarpour@yahoo.com>
Signed-off-by: eunames <eu505@hotmail.com>
Signed-off-by: Ludovic Ortega <ludovic.ortega@adminafk.fr>
Signed-off-by: eunames <eu505@hotmail.com>
@eunames eunames force-pushed the enhancement/add_imagePullSecret branch from b86d012 to 0e3b4e2 Compare April 24, 2024 07:26
@TheBigLee
Copy link
Contributor

@eunames Could you rebase your branch so we can have a look at the change again and merge it if everything looks good?

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

Successfully merging this pull request may close these issues.

Allow to set imagePullSecrets for k8up-io/k8up