-
Notifications
You must be signed in to change notification settings - Fork 230
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
Should not be able to add the same secret or config map to an app twice #2226
Comments
@spadgett do you mean the in the DC's Environment Variables section ? Or BC's Build Secrets section ? |
@jeff-phillips-18 mind if I take this one ? |
go for it @jhadvig |
@jhadvig I mean on the secret or config map page. There's an "Add to Application" button. |
@spadgett what if the application contains two containers, where one of them already contains the secret/configMap... shouldn't we show the application if at least one of its containers doesn't contain the secret/configMap ? |
Good question. I think so. |
Automatic merge from submit-queue. Should not be able to add the same secret or config map to an application twice The `Add to Application` modal will only show applications which DC's won't contain the secret/configMap that user wants to add. In case one of the containers won't contain the secret/configMap the application will be shown in the dropbox, but the ref will be added only to the containers that don't contain the it. Unfortunately if, in case of multiple container DC, user chooses to specify a specific container, the checkbox of the container that already contains the ref wont be pre-selected, cause that could mislead the user that by unchecking the checkbox he could remove the ref from that specific container. Other option would be to pre-select the checkbox and also disable it, but that would require add the disable logic to the `select-container` directive. Wasn't sure if we need that ATM... @jeff-phillips-18 @spadgett PTAL Closes #2226
It's doesn't make sense to add the same secret twice.
cc @cdcabrera
The text was updated successfully, but these errors were encountered: