Thanks to visit codestin.com
Credit goes to github.com

Skip to content

imagePullSecrets is a top-level attribute of ServiceAccounts - #239

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

cognifloyd merged 2 commits into
StackStorm:masterfrom
cognifloyd:fix-sa-pullsecret

Conversation

@cognifloyd

Copy link
Copy Markdown
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
Copy Markdown
Member Author

@armab We need this one for the next release.

@arm4b arm4b left a comment

Copy link
Copy Markdown
Member

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
Copy Markdown
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

arm4b commented Sep 7, 2021

Copy link
Copy Markdown
Member

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