nextcloud / nextcloud/helm

Feature: Pin a specific `nginx` tag

Open
#509 2 comments 1 reaction 0 assignees View on GitHub

Nobody has claimed this yet.

enhancement good first issue
Dominant language
Go Template
Stars
534
Forks
314
Avg merge
42m
Merged PRs (30d)
1

Description

Description of the change

Currently nginx:alpine is used which is bad practice, as the image will never be updated after the initial pull. I suggest that rather a specific nginx version should be used, which would be updated regularly.

Benefits

  • Users wouldn't need to manually check of updates of nginx and manually overwrite the nginx tag.

  • Tests of this Helm chart would be more meaningful as the nginx version that is tested against is what would also be running on enduser's systems.

Possible drawbacks

  • There would be an additional maintenance effor of keeping the version up-to-date in this chart.

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Search the Helm chart for the current nginx:alpine reference and inspect the surrounding image configuration. Confirm how the chart tests exercise that image, then use a specific nginx version so the tested and deployed image are aligned; done means the floating tag is no longer used.

Written by the indexing model from the issue text.

Assessment

Tech stack
helm, kubernetes, nginx
Domain
devops, infrastructure
Issue type
Feature
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.