Skip to content

Bug 1502914 - Compact empty envFrom entries#2275

Merged
openshift-merge-robot merged 1 commit intoopenshift:masterfrom
spadgett:fix-edit-env
Oct 17, 2017
Merged

Bug 1502914 - Compact empty envFrom entries#2275
openshift-merge-robot merged 1 commit intoopenshift:masterfrom
spadgett:fix-edit-env

Conversation

@spadgett
Copy link
Member

Don't include empty envFrom entries when updating environment variables. This causes a validation failure.

Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1502914

@jwforres PTAL

@openshift-ci-robot openshift-ci-robot added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Oct 17, 2017
Don't include empty `envFrom` entries when updating environment
variables. This causes a validation failure.

Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1502914
@jwforres
Copy link
Member

/lgtm

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Oct 17, 2017
@openshift-merge-robot
Copy link
Contributor

Automatic merge from submit-queue.

@openshift-merge-robot openshift-merge-robot merged commit de72b67 into openshift:master Oct 17, 2017
openshift-merge-robot added a commit that referenced this pull request Oct 17, 2017
Automatic merge from submit-queue.

[stage] Bug 1502914 - Compact empty envFrom entries

Don't include empty `envFrom` entries when updating environment variables. This causes a validation failure.

Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1502914

Backport of #2275 to stage branch.
@spadgett spadgett deleted the fix-edit-env branch October 17, 2017 15:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm Indicates that a PR is ready to be merged. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants