-
Notifications
You must be signed in to change notification settings - Fork 3
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
Configuring PAM for K8S prod #159
Conversation
tamas-molnar
commented
Nov 24, 2017
•
edited
edited
- moved read env config to global-configs -see https://github.com/Financial-Times/upp-global-configs/pull/21
- set base url for validation
- for production envs: manually in the app-config
- for team envs: read from deployment param (depends on Jenkins job feature)
- added PAM secrets for production, see lastpass
- updated secrets in K8S CT
@@ -65,20 +65,21 @@ spec: | |||
secretKeyRef: | |||
name: publish-availability-monitor-secrets | |||
key: notifications_push.api_key | |||
{{- $base_url := default .Values.envs.validation_endpoints.base_url .Values.cluster.delivery.url }} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I was wrong on this one.. the parameters should be inverted.
@@ -20,7 +20,5 @@ envs: | |||
methode_article_internal_components_mapper: "__methode-article-internal-components-mapper/map" | |||
video_mapper: "__upp-next-video-mapper/map" | |||
wordpress_mapper: "__wordpress-article-mapper/map" | |||
base_url: "https://upp-k8s-delivery-test-eu.ft.com/" | |||
base_url: "http://upp-prod-delivery.ft.com/" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This needs to be https:
# I had to split the URL into 2 parts because I couldn't find a way to put the exact string \& in the value | ||
# This way I moved handling that value in the template chart where it seems it doesn't have a problem with it. | ||
notifications_push_url: "/content/notifications-push?monitor=true" | ||
notifications_pursh_url_param2: "type=all" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There's a typo in "push"
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
good catch!