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

Update k8-configmap.yaml #4

Open
wants to merge 1 commit into
base: dev
Choose a base branch
from
Open

Update k8-configmap.yaml #4

wants to merge 1 commit into from

Conversation

paulalbert1
Copy link
Contributor

@paulalbert1 paulalbert1 commented May 9, 2023

Hi @rar4026 - These values should not be hard-coded. We need these data to be populated in Configmap for the Python script to execute. I've manually updated these values in ConfigMap for now.

Note that these data are in two different places where we have these values: Secrets and Configmap. Maybe we should just keep them in Secrets going forward. If this is the case, you can disregard this PR.
Screenshot 2023-05-09 at 3 29 41 PM

@paulalbert1 paulalbert1 requested a review from rar4026 May 9, 2023 20:24
@rar4026
Copy link
Contributor

rar4026 commented May 10, 2023

Hi @rar4026 - These values should not be hard-coded. We need these data to be populated in Configmap for the Python script to execute. I've manually updated these values in ConfigMap for now.

Note that these data are in two different places where we have these values: Secrets and Configmap. Maybe we should just keep them in Secrets going forward. If this is the case, you can disregard this PR. Screenshot 2023-05-09 at 3 29 41 PM

I see, actually I missed this configmap values, that is fine let's keep the values in configmap itself, while we were working on the cron job, i manually added the values to secrets. I'll remove them again although the values remain unused from secrets and are redundant.
Now this all makes sense. Thank you !

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