<!--
Before you open the request please review the following guidelines and tips to help it be more easily integrated:
- Describe the scope of your change - i.e. what the change does.
- Describe any known limitations with your change.
- Please run any tests or examples that can exercise your modified code.
Thank you for contributing! We will try to review, test and integrate the change as soon as we can.
-->
### Description of the change
<!-- Describe the scope of your change - i.e. what the change does. -->
### Benefits
<!-- What benefits will be realized by the code change? -->
### Possible drawbacks
<!-- Describe any known limitations with your change -->
### Applicable issues
<!-- Enter any applicable Issues here (You can reference an issue using #). Please remove this section if there is no referenced issue. -->
- Fixes #
### Additional information
<!-- If there's anything else that's important and relevant to your pull request, mention that information here. Please remove this section if it remains empty. -->
### ⚠ BREAKING
<!-- If there's a breaking change, please shortly describe in which way users are affected and how they can mitigate it. If there are no breakings, please remove this section. -->
### Checklist
<!-- [Place an '[X]' (no spaces) in all applicable fields. Please remove unrelated fields.] -->
- [ ] Parameters are documented in the `values.yaml` and added to the `README.md` using [readme-generator-for-helm](https://github.com/bitnami-labs/readme-generator-for-helm)
- [ ] Breaking changes are documented in the `README.md`
- [ ] Helm templating unittests are added (required when changing anything in `templates` folder)
- [ ] Bash unittests are added (required when changing anything in `scripts` folder)
- [ ] All added template resources MUST render a namespace in metadata
Reviewed-on: https://gitea.com/gitea/helm-actions/pulls/116
Reviewed-by: Lunny Xiao <[email protected]>
Co-authored-by: Daan Selen <[email protected]>
Co-committed-by: Daan Selen <[email protected]>
Once: https://gitea.com/gitea/act_runner/pulls/738
Is merged.
Then this can be applied to specify timezone for example for cases such as:
Right now the correct time has to be retrieved with a ENV variable in the cmd: `git commit -m "Update custom ISO: $(TZ=Europe/Amsterdam date +'%Y-%m-%d %H:%M')" || echo "No changes to commit"`
Co-authored-by: Daan Selen <[email protected]>
Reviewed-on: https://gitea.com/gitea/helm-actions/pulls/38
Reviewed-by: techknowlogick <[email protected]>
Co-authored-by: Daan Selen <[email protected]>
Co-committed-by: Daan Selen <[email protected]>
### Description of the change
replace the netcat command with wget commands.
### Benefits
Less noise in the log(s)
### Possible drawbacks
<!-- Describe any known limitations with your change -->
No limitations. As far as I am aware.
- Fixes #
https://gitea.com/gitea/helm-actions/pulls/23#issuecomment-1027779
### Additional information
As the PR Title suggests, it replaces the netcat command to a wget command which makes the noise in the log stop.
### Checklist
- [X] Parameters are documented in the `values.yaml` and added to the `README.md` using [readme-generator-for-helm](https://github.com/bitnami-labs/readme-generator-for-helm)
- [X] Breaking changes are documented in the `README.md`
- [X] Helm templating unittests are added (required when changing anything in `templates` folder)
- [X] Bash unittests are added (required when changing anything in `scripts` folder)
- [X] All added template resources MUST render a namespace in metadata
Co-authored-by: Daan Selen <[email protected]>
Reviewed-on: https://gitea.com/gitea/helm-actions/pulls/26
Reviewed-by: ChristopherHX <[email protected]>
Co-authored-by: Daan Selen <[email protected]>
Co-committed-by: Daan Selen <[email protected]>
daanselen
