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

Syncing latest changes from master for noobaa-operator #7

Merged
merged 24 commits into from
Apr 2, 2024

Conversation

df-build-team
Copy link

PR containing the latest commits from master branch

@df-build-team df-build-team requested a review from a team March 1, 2024 23:33
shirady and others added 19 commits March 3, 2024 09:30
Remove the certificate fix just for GCP as it was causing reflection
errors (See noobaa#1312 for more info).

Signed-off-by: Ben <[email protected]>
[DB Upgrade] Fixing returned status and adding prints
Bump dependencies

Signed-off-by: liranmauda <[email protected]>
Updating
go lang version to v1.21
controller-gen to v0.14.0
deepcopy-gen to v0.29.3

Signed-off-by: Vinayakswami Hariharmath <[email protected]>
Adding a sidecar container which will run rsyslod and logrotate
It will also be running a log segreagate script.

Signed-off-by: Ashish Pandey <[email protected]>
bucket_logging: Bucket log processing in side car container
This patch provides the support for azure keyvault.

We are using "libopenstorage/secrets" as the wrapper
package to integrate with different kms and package
provides the abstraction over several kms. It also
provides the integraton support for azure and helps
communication with azure key vault.

We are required to provide the definition for house
keeping calls registered calls with libopenstorage/secrets.

"libopenstorage/secrets" does the creation of client
handle based on the details provided in configmap.
The certificate details present in the secret are
preserved inside a temp file and used to establish
the connection with azure key vault as of now.

Below are the connection details that are going to be
populated on Noobaa CR by `noobaa_system_reconciler`
at ocs side and this is the ocs code where connectiondetails
on Noobaa CR are built: https://github.com/red-hat-storage/ocs-operator/blob/2d082fc4c1ac4cec961406053cece448f4b07684/controllers/storagecluster/noobaa_system_reconciler.go#L249

ex: configmap data:
```
data:
  AZURE_CERT_SECRET_NAME: azure-ocs-ffwc9o1j
  AZURE_CLIENT_ID: az-client-id1
  AZURE_TENANT_ID: az-tenant-id1
  AZURE_VAULT_URL: az-valut-url1
  KMS_PROVIDER: azure-kv
  KMS_SERVICE_NAME: kms-conn-azure1
```

Signed-off-by: Vinayakswami Hariharmath <[email protected]>
Copy link

openshift-ci bot commented Apr 2, 2024

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: df-build-team, Nikhil-Ladha

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@Nikhil-Ladha Nikhil-Ladha merged commit 5b28d29 into release-4.16 Apr 2, 2024
28 of 29 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants