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.

Sign up for GitHub

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

Jump to bottom

Installation manager doesn't store empty codenvy property #512

Closed
dmytro-ndp opened this issue Aug 3, 2016 · 0 comments
Closed

Installation manager doesn't store empty codenvy property #512

dmytro-ndp opened this issue Aug 3, 2016 · 0 comments
Assignees
@dmytro-ndp
Labels
kind/bug status/in-progress team/enterprise
Milestone
4.7.0

Comments

@dmytro-ndp
Copy link
Contributor

Reproduction Steps:

  • execute command:

codenvy config docker_registry_credentials ""

Expected behavior:

[CODENVY] Do you want to update Codenvy property 'docker_registry_credentials' with new value ''? [y/N] y
docker_registry_credentials=

Observed behavior:
installation manager doesn't take into account empty value:

docker_registry_credentials=registry1.url=http://host
dmytro-ndp pushed a commit that referenced this issue Aug 3, 2016
(#512) add ability to store empty codenvy properties
bd212d3 
Signed-off-by: Dmytro Nochevnov <[email protected]>
@dmytro-ndp dmytro-ndp self-assigned this Aug 3, 2016
@dmytro-ndp dmytro-ndp changed the title It's impossible to store empty codenvy property by 'codenvy config' command of installation manager Installation manager doesn't store empty codenvy property Aug 3, 2016
@dmytro-ndp dmytro-ndp added this to the 4.7.0 milestone Aug 3, 2016
@JamesDrummond JamesDrummond mentioned this issue Aug 21, 2016
Closed
26 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@dmytro-ndp dmytro-ndp

Labels
kind/bug status/in-progress team/enterprise
Projects
None yet
Milestone
4.7.0
Development

No branches or pull requests
2 participants
@dmytro-ndp @vkuznyetsov