Skip to content

imagePullSecrets is a top-level attribute of ServiceAccounts#239

Merged
cognifloyd merged 2 commits intoStackStorm:masterfrom
cognifloyd:fix-sa-pullsecret
Sep 7, 2021
Merged

imagePullSecrets is a top-level attribute of ServiceAccounts#239
cognifloyd merged 2 commits intoStackStorm:masterfrom
cognifloyd:fix-sa-pullsecret

Conversation

@cognifloyd
Copy link
Member

resolves #227

@pull-request-size pull-request-size bot added the size/XS PR that changes 0-9 lines. Quick fix/merge. label Sep 6, 2021
@cognifloyd cognifloyd added bug Something isn't working RFR labels Sep 6, 2021
@cognifloyd cognifloyd requested a review from arm4b September 6, 2021 14:36
@cognifloyd cognifloyd enabled auto-merge September 6, 2021 14:36
@cognifloyd
Copy link
Member Author

@armab We need this one for the next release.

Copy link
Member

@arm4b arm4b left a comment

Choose a reason for hiding this comment

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

Looks good, but we'll still need a Changelog here 👍

Otherwise, it will be hard for people to fast-track the issues they experienced that might be silently fixed by us.

@cognifloyd
Copy link
Member Author

This fixes the syntax for #196 which has not been released yet. No one had been able to use this feature yet since it has been broken since it was introduced. Why do we need another entry?

@arm4b
Copy link
Member

arm4b commented Sep 7, 2021

There are folks who rely on the latest master chart and so the Changelog is always preferred as a way of tracking and sharing the progress and fixes.

@cognifloyd cognifloyd merged commit 4d19ef9 into StackStorm:master Sep 7, 2021
@cognifloyd cognifloyd removed the RFR label Sep 8, 2021
@cognifloyd cognifloyd deleted the fix-sa-pullsecret branch November 11, 2021 18:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working size/XS PR that changes 0-9 lines. Quick fix/merge.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Fix indentation for ServiceAccount pullSecrets

2 participants