Skip to content

Fix for refreshing of objectstorage-keyfile#10

Merged
Cryptophobia merged 2 commits into
masterfrom
issue/fix-objecstorage-keyfile-in-namespaces
Oct 20, 2020
Merged

Fix for refreshing of objectstorage-keyfile#10
Cryptophobia merged 2 commits into
masterfrom
issue/fix-objecstorage-keyfile-in-namespaces

Conversation

@Cryptophobia
Copy link
Copy Markdown
Member

@Cryptophobia Cryptophobia commented Mar 12, 2018

This is the fix for the bug #9 which affects objectstorage-keyfile secrets in application namespaces. With this fix, the controller will copy the objecstorage-keyfile from the main deis workflow namespace and existing buildpack applications will have fresh objectstorage-keyfile secret when their configuration changes or when they are deployed by the controller because the controller will now check if the objectstorage-keyfile has changed. New buildpack applications always get the latest objectstorage-keyfile so that has not changed. 👍

All tests should pass.
It was tested in a live production workflow build with many applications.

Copy link
Copy Markdown
Member

@kingdonb kingdonb left a comment

Choose a reason for hiding this comment

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

Approval

Signed-off-by: Cryptophobia <aouzounov@gmail.com>
@Cryptophobia Cryptophobia force-pushed the issue/fix-objecstorage-keyfile-in-namespaces branch from 18c75ce to ec87381 Compare October 20, 2020 20:23
@Cryptophobia Cryptophobia merged commit 3aab23d into master Oct 20, 2020
duanhongyi added a commit to duanhongyi/controller that referenced this pull request Jan 3, 2021
feat(controller): add a volume command
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.

2 participants