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

feat: add cosi helm chart #1529

Merged
merged 2 commits into from
Dec 19, 2024
Merged

feat: add cosi helm chart #1529

merged 2 commits into from
Dec 19, 2024

Conversation

takirala
Copy link
Contributor

@takirala takirala commented Dec 16, 2024

What type of PR is this?

Adding a cosi helm chart which will be consumed by CAREN
https://jira.nutanix.com/browse/NCN-104827

What this PR does/ why we need it:

~/git/output/daily ❯ k get crds|grep objectstorage                                                                                                13:53:53
bucketaccessclasses.objectstorage.k8s.io                                        2024-12-18T21:47:13Z
bucketaccesses.objectstorage.k8s.io                                             2024-12-18T21:47:13Z
bucketclaims.objectstorage.k8s.io                                               2024-12-18T21:47:13Z
bucketclasses.objectstorage.k8s.io                                              2024-12-18T21:47:13Z
buckets.objectstorage.k8s.io                                                    2024-12-18T21:47:13Z
~/git/output/daily ❯ k get po -ncontainer-object-storage-system                                                                                   13:54:02
NAME                                                   READY   STATUS    RESTARTS   AGE
container-object-storage-controller-7f9f89fd45-qkfcg   1/1     Running   0          4m43s

Tested this on daily cluster:

apiVersion: helm.toolkit.fluxcd.io/v2
kind: HelmRelease
metadata:
  annotations:
    kubectl.kubernetes.io/last-applied-configuration: |
      {"apiVersion":"helm.toolkit.fluxcd.io/v2beta1","kind":"HelmRelease","metadata":{"annotations":{},"name":"takirala","namespace":"default"},"spec":{"chart":{"spec":{"chart":"cosi","sourceRef":{"kind":"HelmRepository","name":"takirala"},"version":"0.0.1-alpha.1"}},"interval":"10m","targetNamespace":"kommander"}}
    reconcile.fluxcd.io/requestedAt: "2024-12-18T13:48:35.547345-08:00"
  creationTimestamp: "2024-12-18T21:46:58Z"
  finalizers:
  - finalizers.fluxcd.io
  generation: 2
  name: takirala
  namespace: default
  resourceVersion: "1612435"
  uid: 9672dc2d-8762-4543-8aae-01cd6cecda0c
spec:
  chart:
    spec:
      chart: cosi
      reconcileStrategy: ChartVersion
      sourceRef:
        kind: HelmRepository
        name: takirala
      version: 0.0.1-alpha.1
  interval: 10m
  targetNamespace: kommander
status:
  conditions:
  - lastTransitionTime: "2024-12-18T21:49:26Z"
    message: Helm install succeeded for release kommander/kommander-takirala.v1 with
      chart [email protected]
    observedGeneration: 2
    reason: InstallSucceeded
    status: "True"
    type: Ready
  - lastTransitionTime: "2024-12-18T21:49:26Z"
    message: Helm install succeeded for release kommander/kommander-takirala.v1 with
      chart [email protected]
    observedGeneration: 2
    reason: InstallSucceeded
    status: "True"
    type: Released
  helmChart: default/default-takirala
  history:
  - appVersion: 0.0.1-alpha.1
    chartName: cosi
    chartVersion: 0.0.1-alpha.1
    configDigest: sha256:e3b0c44298fc1c149afbf4c8996fb92427ae41e4649b934ca495991b7852b855
    digest: sha256:96ed7b0aee819636016d9492b402dacc74c3358ed77743576f9ee87c653c39f4
    firstDeployed: "2024-12-18T21:49:24Z"
    lastDeployed: "2024-12-18T21:49:24Z"
    name: kommander-takirala
    namespace: kommander
    status: deployed
    version: 1
  lastAttemptedConfigDigest: sha256:e3b0c44298fc1c149afbf4c8996fb92427ae41e4649b934ca495991b7852b855
  lastAttemptedGeneration: 2
  lastAttemptedReleaseAction: install
  lastAttemptedRevision: 0.0.1-alpha.1
  lastHandledReconcileAt: "2024-12-18T13:48:35.547345-08:00"
  observedGeneration: 2
  storageNamespace: default

Which issue(s) this PR fixes:

Special notes for your reviewer:

Does this PR introduce a user-facing change?:


Checklist

  • If a chart is changed, the chart version is correctly incremented.
  • The commit message explains the changes and why are needed.
  • The code builds and passes lint/style checks locally.
  • The relevant subset of integration tests pass locally.
  • The core changes are covered by tests.
  • The documentation is updated where needed.

Signed-off-by: Tarun Gupta Akirala <[email protected]>
@takirala takirala marked this pull request as ready for review December 18, 2024 21:55
@takirala takirala requested review from a team as code owners December 18, 2024 21:55
@takirala takirala merged commit 4bcc9dd into mesosphere:master Dec 19, 2024
5 checks passed
@takirala takirala deleted the tga/cosi branch December 19, 2024 18:32
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