Skip to content
This repository was archived by the owner on Mar 16, 2024. It is now read-only.

Conversation

@iwilltry42
Copy link
Contributor

Ref #1974

So far, the registry was backed by an EmptyDir, so restarts/redeployments, e.g. as part of an upgrade cleared the registry and we would end up with image resources backed by nothing in the registry.
With this PR we get a basic 10Gi PVC backing the registry if there's a default storage class defined upon installation.
As discussed, there are not configuration options available.

Checklist

  • The title of this PR would make a good line in Acorn's Release Note's Changelog
  • The title of this PR ends with a link to the main issue being address in parentheses, like: This is a title (#1216). Here's an example
  • All relevant issues are referenced in the PR description. NOTE: don't use GitHub keywords that auto-close issues
  • Commits follow contributing guidance
  • Automated tests added to cover the changes. If tests couldn't be added, an explanation is provided in the Verification and Testing section
  • Changes to user-facing functionality, API, CLI, and upgrade impacts are clearly called out in PR description
  • PR has at least two approvals before merging (or a reasonable exception, like it's just a docs change)

Signed-off-by: Thorsten Klein <tk@thklein.io>
@iwilltry42 iwilltry42 merged commit 6db89ac into acorn-io:main Jul 28, 2023
@iwilltry42 iwilltry42 deleted the iwilltry42/issue1974 branch August 14, 2023 13:05
cloudnautique pushed a commit to cloudnautique/runtime that referenced this pull request Sep 28, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants