Skip to content

Registry for postgresql initContainer#262

Closed
fdelucchijr wants to merge 3 commits intonextcloud:mainfrom
fdelucchijr:patch-1
Closed

Registry for postgresql initContainer#262
fdelucchijr wants to merge 3 commits intonextcloud:mainfrom
fdelucchijr:patch-1

Conversation

@fdelucchijr
Copy link
Copy Markdown

Signed-off-by: Fernando De Lucchi [email protected]

Pull Request

Description of the change

Just parses registry from values.yaml Postgres image key

Signed-off-by: Fernando De Lucchi <[email protected]>
fdelucchijr and others added 2 commits August 10, 2022 22:10
You’re right! I didn’t think about it!

Co-authored-by: Michael <[email protected]>
Signed-off-by: Fernando De Lucchi <[email protected]>
Signed-off-by: Fernando De Lucchi <[email protected]>
@MichaelSp
Copy link
Copy Markdown
Contributor

btw: @fdelucchijr I'm not a maintainer here

@fdelucchijr
Copy link
Copy Markdown
Author

btw: @fdelucchijr I'm not a maintainer here

15 days already had passed, this repo is still maintained?

@jessebot jessebot self-requested a review April 10, 2023 17:37
@jessebot
Copy link
Copy Markdown
Collaborator

Hi, sorry for the delay! This repo is still maintained by volunteers when they have the time.

Copy link
Copy Markdown
Collaborator

@jessebot jessebot left a comment

Choose a reason for hiding this comment

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

You need to bump the helm chart version when you make a change as per the checklist in the pull request template. You should also update the README.md and values.yaml with any new parameters you are introducing.

@jessebot
Copy link
Copy Markdown
Collaborator

jessebot commented Sep 5, 2023

@fdelucchijr if you're still available, could you please bump the helm chart version?

@jessebot
Copy link
Copy Markdown
Collaborator

closing this as it is resolved in #471

@jessebot jessebot closed this Dec 15, 2023
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.

3 participants