Skip to content

fix(secgroups): do not null out ingress & egress#128

Merged
metral merged 2 commits into
masterfrom
metral/fix-secgroup-rule-defaults
May 8, 2019
Merged

fix(secgroups): do not null out ingress & egress#128
metral merged 2 commits into
masterfrom
metral/fix-secgroup-rule-defaults

Conversation

@metral

@metral metral commented May 4, 2019

Copy link
Copy Markdown
Contributor

Fixes #127

  • Note: This PR reverses the default null values used for the
    ingress and egress in-line rules of the secgroups, introduced in v0.18.3.
    The null default was required to move to standalone secgroup rules.
  • Upgrade Path - This is a breaking change unless you do the following steps:
    • If using >= v0.18.3: update using the typical package update path.
    • If using <= v0.18.2:
      1. First, update your cluster from using your current version to v0.18.4.
      2. Next, update your cluster from v0.18.4 to v0.18.5 (or higher) using the typical package update path.

@metral metral requested a review from lukehoban May 4, 2019 20:35
@naineel

naineel commented May 7, 2019

Copy link
Copy Markdown
Contributor

Just hit this myself

@lukehoban lukehoban left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

Comment thread CHANGELOG.md Outdated
- Note: This PR reverses the default null values used for the
  ingress and egress in-line rules of the secgroups, introduced in `v0.18.3`.
  The null default was required to move to standalone secgroup rules.
- Upgrade Path - This is a breaking change **unless** you do the
following steps:
  - If using >= `v0.18.3`: update using the typical package update path.
  - If using <= `v0.18.2`:
    1. First, update your cluster from using your current version to `v0.18.4`.
    1. Next, update your cluster from `v0.18.4` to `v0.18.5` (or higher) using the typical package update path.
@metral metral force-pushed the metral/fix-secgroup-rule-defaults branch from bd17a63 to 8ccd4d1 Compare May 8, 2019 21:12
@metral metral merged commit d345f7b into master May 8, 2019
@pulumi-bot pulumi-bot deleted the metral/fix-secgroup-rule-defaults branch May 8, 2019 22:03
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.

Defaulting the secgroups in-line rules to null creates empty secgroups on a cluster replacement

3 participants