Skip to content

Commit

Permalink
Opt-out useryaml when using usersync
Browse files Browse the repository at this point in the history
  • Loading branch information
flashguerdon committed Jul 30, 2024
1 parent f08535b commit b35d6e2
Show file tree
Hide file tree
Showing 6 changed files with 10 additions and 8 deletions.
2 changes: 1 addition & 1 deletion docs/fence_usersync_job.md
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
# Fence Usersync CronJob

If `.Values.usersync.usersync` is set to true, the Fence usersync-cron.yaml will be deployed to the cluster.
If `.Values.usersync.usersync` is set to true, the Fence usersync-cron.yaml will be deployed to the cluster, otherwise useryaml job will be deployed instead of usersync.

User lists can be synced from three sources:

Expand Down
2 changes: 1 addition & 1 deletion helm/common/Chart.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@ type: library
# This is the chart version. This version number should be incremented each time you make changes
# to the chart and its templates, including the app version.
# Versions are expected to follow Semantic Versioning (https://semver.org/)
version: 0.1.11
version: 0.1.12

# This is the version number of the application being deployed. This version number should be
# incremented each time you make changes to the application. Versions are not expected to
Expand Down
4 changes: 2 additions & 2 deletions helm/fence/Chart.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@ type: application
# This is the chart version. This version number should be incremented each time you make changes
# to the chart and its templates, including the app version.
# Versions are expected to follow Semantic Versioning (https://semver.org/)
version: 0.1.19
version: 0.1.20

# This is the version number of the application being deployed. This version number should be
# incremented each time you make changes to the application. Versions are not expected to
Expand All @@ -24,7 +24,7 @@ appVersion: "master"

dependencies:
- name: common
version: 0.1.11
version: 0.1.12
repository: file://../common
- name: postgresql
version: 11.9.13
Expand Down
2 changes: 2 additions & 0 deletions helm/fence/templates/useryaml-job.yaml
Original file line number Diff line number Diff line change
@@ -1,3 +1,4 @@
{{ if not .Values.usersync.usersync }}
kind: ConfigMap
apiVersion: v1
metadata:
Expand Down Expand Up @@ -47,3 +48,4 @@ spec:
# can be removed once this is merged: https://github.com/uc-cdis/fence/pull/1096
fence-create sync --arborist http://arborist-service --yaml /var/www/fence/user.yaml
restartPolicy: OnFailure
{{ end }}
2 changes: 1 addition & 1 deletion helm/fence/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -100,7 +100,7 @@ externalSecrets:
# -- (map) Configuration options for usersync cronjob.
usersync:
# -- (bool) Whether to run Fence usersync or not.
usersync: true
usersync: false
# -- (string) The cron schedule expression to use in the usersync cronjob. Runs every 30 minutes by default.
schedule: "*/30 * * * *"
# -- (string) To set a custom image for pulling the user.yaml file from S3. Default is the Gen3 Awshelper image.
Expand Down
6 changes: 3 additions & 3 deletions helm/gen3/Chart.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@ dependencies:
repository: "file://../aws-es-proxy"
condition: aws-es-proxy.enabled
- name: common
version: 0.1.11
version: 0.1.12
repository: file://../common
- name: etl
version: 0.1.1
Expand All @@ -36,7 +36,7 @@ dependencies:
repository: "file://../frontend-framework"
condition: frontend-framework.enabled
- name: fence
version: 0.1.19
version: 0.1.20
repository: "file://../fence"
condition: fence.enabled
- name: guppy
Expand Down Expand Up @@ -128,7 +128,7 @@ type: application
# This is the chart version. This version number should be incremented each time you make changes
# to the chart and its templates, including the app version.
# Versions are expected to follow Semantic Versioning (https://semver.org/)
version: 0.1.38
version: 0.1.39

# This is the version number of the application being deployed. This version number should be
# incremented each time you make changes to the application. Versions are not expected to
Expand Down

0 comments on commit b35d6e2

Please sign in to comment.