Skip to content

chore(dagger): Update dagger helm chart to have conditional configuration for gcp environment. #132

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

Merged
merged 3 commits into from
Feb 18, 2025

Conversation

rajuGT
Copy link

@rajuGT rajuGT commented Feb 17, 2025

No description provided.

@rajuGT rajuGT merged commit 032a94f into dagger_ha_convention_change Feb 18, 2025
rajuGT added a commit that referenced this pull request Mar 3, 2025
* chore(dagger): dagger/flink job bucket url convention change
Existing:
Checkpoint: g-godata-id-dagger-checkpoints/flink-checkpoints/{{ .Values.name }}
Savepoint: g-godata-id-dagger-checkpoints/flink-savepoints/{{ .Values.name }}
high-availability.storageDir: g-godata-id-dagger-checkpoints/ha

New:
Checkpoint: g-gojek-id-dagger-ha/checkpoints/{{ .Values.name }}
Savepoint: g-gojek-id-dagger-ha/savepoints/{{ .Values.name }}
high-availability.storageDir: g-gojek-id-dagger-ha/k8s-ha

* chore(dagger): chart compatibility w.r.t GCS setup

* chore(dagger): remove dagger chart default values for checkpoint,savepoint, k8s url

* chore(dagger): Update dagger helm chart to have conditional configuration for gcp environment. (#132)

* chore(dagger): Update dagger helm chart to have conditional configuration for gcp environment.

* address MR comment - renaming csp to cloud_provider

---------

Co-authored-by: rajuGT <raju.gt@gojek.com>

* feat(dagger): add ali credential configmap

* add config credential-provider EnvironmentVariableCredentialProvider for
both ali and tcc.

* code reformat

* add fs.oss.endpoint and change dagger-user-crds from configRef to secretRef

---------

Co-authored-by: rajuGT <raju.gt@gojek.com>
Co-authored-by: Femi Novia Lina <feminovi@gmail.com>
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.

1 participant