Commit Graph

2483 Commits

Author SHA1 Message Date
Kjeld Schouten
5e65517b88 Merge branch 'main' into verifymore 2023-06-07 10:56:10 +02:00
Kjeld Schouten
fc4f65fae6 Chore(common): bump common common-12.12.4 2023-06-07 10:33:12 +02:00
Kjeld Schouten
94c84c2245 fix(common): set rotation-policy on certificate class 2023-06-07 10:31:55 +02:00
Kjeld Schouten
30fc71519c fix(common): always rotate cert-manager privatekeys on ingress 2023-06-07 10:30:49 +02:00
Kjeld Schouten
fcea6952fb Update library/common/values.yaml 2023-06-07 09:45:18 +02:00
Kjeld Schouten
533656db23 Merge branch 'main' into verifymore 2023-06-07 09:40:29 +02:00
Kjeld Schouten
5f65dd5178 Apply suggestions from code review 2023-06-07 09:40:04 +02:00
Kjeld Schouten
ed0d6c09dd feat(common): allow for non-expanded service names (#459)
**Description**
Primarily operators sometimes have hardcoded svc names that do not allow
for appending the full-name

**⚙️ Type of change**

- [x] ⚙️ Feature/App addition
- [x] 🪛 Bugfix
- [ ] ⚠️ Breaking change (fix or feature that would cause existing
functionality to not work as expected)
- [ ] 🔃 Refactor of current code

**🧪 How Has This Been Tested?**
<!--
Please describe the tests that you ran to verify your changes. Provide
instructions so we can reproduce. Please also list any relevant details
for your test configuration
-->

**📃 Notes:**
<!-- Please enter any other relevant information here -->

**✔️ Checklist:**

- [x] ⚖️ My code follows the style guidelines of this project
- [x] 👀 I have performed a self-review of my own code
- [x] #️⃣ I have commented my code, particularly in hard-to-understand
areas
- [x] 📄 I have made corresponding changes to the documentation
- [x] ⚠️ My changes generate no new warnings
- [x] 🧪 I have added tests to this description that prove my fix is
effective or that my feature works
- [x] ⬆️ I increased versions for any altered app according to semantic
versioning

** App addition**

If this PR is an app addition please make sure you have done the
following.

- [ ] 🪞 I have opened a PR on
[truecharts/containers](https://github.com/truecharts/containers) adding
the container to TrueCharts mirror repo.
- [ ] 🖼️ I have added an icon in the Chart's root directory called
`icon.png`

---

_Please don't blindly check all the boxes. Read them and only check
those that apply.
Those checkboxes are there for the reviewer to see what is this all
about and
the status of this PR with a quick glance._
common-12.12.3
2023-06-06 09:12:27 +02:00
Kjeld Schouten-Lebbing
bb1b36d86a prometheus fix name to prometheus-operator 2023-06-05 11:20:27 +02:00
Kjeld Schouten-Lebbing
d5c9ccf9ef revemo old useless crds 2023-06-05 11:09:58 +02:00
Kjeld Schouten-Lebbing
8793281c85 whoopsies 2023-06-05 11:09:30 +02:00
Kjeld Schouten
845330ae74 Update library/common/templates/spawner/_portal.tpl
Co-authored-by: Stavros Kois <47820033+stavros-k@users.noreply.github.com>
2023-06-05 10:48:58 +02:00
Kjeld Schouten
6bea7a7c25 Update library/common/templates/lib/util/_chartcontext.tpl
Co-authored-by: Stavros Kois <47820033+stavros-k@users.noreply.github.com>
2023-06-05 10:48:46 +02:00
Kjeld Schouten
2cf57f3da7 Update library/common/templates/lib/util/_verify_operator.tpl
Co-authored-by: Stavros Kois <47820033+stavros-k@users.noreply.github.com>
2023-06-05 10:48:39 +02:00
Kjeld Schouten
50c1ace4f9 Update library/common/templates/lib/util/_verify_operator.tpl
Co-authored-by: Stavros Kois <47820033+stavros-k@users.noreply.github.com>
2023-06-05 10:48:33 +02:00
Kjeld Schouten
ffa0c2714d Update library/common/templates/lib/util/_verify_operator.tpl
Co-authored-by: Stavros Kois <47820033+stavros-k@users.noreply.github.com>
2023-06-05 10:48:21 +02:00
Kjeld Schouten-Lebbing
803b9fab8e remove the CRD thing, one thing at a time 2023-06-05 10:38:34 +02:00
Kjeld Schouten-Lebbing
b4816745cf install -> upgrade for crds 2023-06-05 10:27:32 +02:00
Kjeld Schouten-Lebbing
d27953df81 also update portal and chartContext 2023-06-05 10:04:55 +02:00
Kjeld Schouten-Lebbing
af0336b284 whoops 2023-06-05 09:48:31 +02:00
Kjeld Schouten-Lebbing
6196af3bf3 actually set register to true, instead of assuming it's true when crds are being installed 2023-06-05 09:48:14 +02:00
Kjeld Schouten-Lebbing
adb7c04e42 when CRDs are set to install, assume it's an operator and register none-the less, as we cannot have duplicate crds being installed. 2023-06-05 09:45:10 +02:00
Kjeld Schouten-Lebbing
b556ae6ee9 Adds some hook info 2023-06-05 09:42:15 +02:00
Kjeld Schouten-Lebbing
318c85b1c3 fixes 2023-06-05 09:39:50 +02:00
Kjeld Schouten-Lebbing
968bbcd804 whoopsies 2023-06-05 09:33:30 +02:00
Kjeld Schouten-Lebbing
a632df0dfb feat(common): also verify traefik, prometheus and cnpg automatically 2023-06-05 09:29:25 +02:00
Stavros Kois
70e1d073c7 chore(common): flatten the verify operator code (#456)
**Description**
<!--
Please include a summary of the change and which issue is fixed. Please
also include relevant motivation and context. List any dependencies that
are required for this change.
-->
⚒️ Fixes  # <!--(issue)-->
No changes, just flattens the code for readability.

**⚙️ Type of change**

- [ ] ⚙️ Feature/App addition
- [ ] 🪛 Bugfix
- [ ] ⚠️ Breaking change (fix or feature that would cause existing
functionality to not work as expected)
- [x] 🔃 Refactor of current code

**🧪 How Has This Been Tested?**
<!--
Please describe the tests that you ran to verify your changes. Provide
instructions so we can reproduce. Please also list any relevant details
for your test configuration
-->

**📃 Notes:**
<!-- Please enter any other relevant information here -->

**✔️ Checklist:**

- [x] ⚖️ My code follows the style guidelines of this project
- [x] 👀 I have performed a self-review of my own code
- [x] #️⃣ I have commented my code, particularly in hard-to-understand
areas
- [ ] 📄 I have made corresponding changes to the documentation
- [x] ⚠️ My changes generate no new warnings
- [ ] 🧪 I have added tests to this description that prove my fix is
effective or that my feature works
- [ ] ⬆️ I increased versions for any altered app according to semantic
versioning

** App addition**

If this PR is an app addition please make sure you have done the
following.

- [ ] 🪞 I have opened a PR on
[truecharts/containers](https://github.com/truecharts/containers) adding
the container to TrueCharts mirror repo.
- [ ] 🖼️ I have added an icon in the Chart's root directory called
`icon.png`

---

_Please don't blindly check all the boxes. Read them and only check
those that apply.
Those checkboxes are there for the reviewer to see what is this all
about and
the status of this PR with a quick glance._
2023-06-05 09:08:43 +02:00
TrueCharts-Admin
3d2b854bf2 chore(deps): update renovatebot/github-action action to v38.1.2 2023-06-05 08:06:02 +02:00
Kjeld Schouten
26c8d638f6 fix(common): fix errors when operatorcache not found 2023-06-04 13:01:25 +02:00
Kjeld Schouten
24d3c116c1 Update register-operator-values.yaml 2023-06-04 12:08:08 +02:00
Kjeld Schouten
f08931e9ed Update common_library_tests.yaml 2023-06-04 12:06:42 +02:00
TrueCharts Bot
ca7f755508 chore(deps): update renovatebot/github-action action to v38.1.1 (#454)
This PR contains the following updates:

| Package | Type | Update | Change |
|---|---|---|---|
|
[renovatebot/github-action](https://togithub.com/renovatebot/github-action)
| action | patch | `v38.1.0` -> `v38.1.1` |

---

### Release Notes

<details>
<summary>renovatebot/github-action</summary>

###
[`v38.1.1`](https://togithub.com/renovatebot/github-action/releases/tag/v38.1.1)

[Compare
Source](https://togithub.com/renovatebot/github-action/compare/v38.1.0...v38.1.1)

##### Documentation

- fix wrong image name in action description
([#&#8203;760](https://togithub.com/renovatebot/github-action/issues/760))
([c64d589](c64d589a04))
- rewrite readme
([#&#8203;722](https://togithub.com/renovatebot/github-action/issues/722))
([b3ab89c](b3ab89c14b))
- update references to ghcr.io/renovatebot/renovate to v35.103.0
([1cdcea2](1cdcea2f8b))

##### Continuous Integration

- **deps:** update renovate docker tag to v35.102.4
([#&#8203;757](https://togithub.com/renovatebot/github-action/issues/757))
([299e606](299e606259))
- **deps:** update renovate docker tag to v35.102.7
([8d99b3f](8d99b3f8e5))
- **deps:** update renovate docker tag to v35.103.0
([#&#8203;758](https://togithub.com/renovatebot/github-action/issues/758))
([d2a5dff](d2a5dff249))
- **deps:** update renovate docker tag to v35.104.0
([#&#8203;759](https://togithub.com/renovatebot/github-action/issues/759))
([2fbe967](2fbe9677f9))
- **deps:** update renovate docker tag to v35.105.0
([688c604](688c6047e6))
- **deps:** update renovate docker tag to v35.105.1
([6d591bf](6d591bf74c))
- **deps:** update renovate docker tag to v35.105.5
([#&#8203;761](https://togithub.com/renovatebot/github-action/issues/761))
([58a0a45](58a0a45663))

##### Miscellaneous Chores

- **deps:** update dependency
[@&#8203;types/node](https://togithub.com/types/node) to v16.18.32
([95d7569](95d75696d0))
- **deps:** update dependency
[@&#8203;types/node](https://togithub.com/types/node) to v16.18.33
([6b64622](6b64622055))
- **deps:** update dependency
[@&#8203;types/node](https://togithub.com/types/node) to v16.18.34
([a506cfd](a506cfd975))
- **deps:** update dependency eslint to v8.41.0
([c36e2e1](c36e2e1183))
- **deps:** update linters to v5.59.7
([a3381cf](a3381cfba7))

##### Build System

- **deps:** update dependency
[@&#8203;semantic-release/github](https://togithub.com/semantic-release/github)
to v8.0.8
([f2c26f0](f2c26f06cc))

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined),
Automerge - At any time (no schedule defined).

🚦 **Automerge**: Enabled.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the
rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update
again.

---

- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check
this box

---

This PR has been generated by [Renovate
Bot](https://togithub.com/renovatebot/renovate).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNS4xMTAuMCIsInVwZGF0ZWRJblZlciI6IjM1LjExMC4wIiwidGFyZ2V0QnJhbmNoIjoibWFpbiJ9-->
common-12.12.2
2023-06-04 11:12:54 +02:00
Stavros Kois
eff17a3ae1 Set ndots to 1 on manifest manager too (#453)
**Description**
<!--
Please include a summary of the change and which issue is fixed. Please
also include relevant motivation and context. List any dependencies that
are required for this change.
-->
⚒️ Fixes  # <!--(issue)-->

**⚙️ Type of change**

- [ ] ⚙️ Feature/App addition
- [ ] 🪛 Bugfix
- [ ] ⚠️ Breaking change (fix or feature that would cause existing
functionality to not work as expected)
- [ ] 🔃 Refactor of current code

**🧪 How Has This Been Tested?**
<!--
Please describe the tests that you ran to verify your changes. Provide
instructions so we can reproduce. Please also list any relevant details
for your test configuration
-->

**📃 Notes:**
<!-- Please enter any other relevant information here -->

**✔️ Checklist:**

- [ ] ⚖️ My code follows the style guidelines of this project
- [ ] 👀 I have performed a self-review of my own code
- [ ] #️⃣ I have commented my code, particularly in hard-to-understand
areas
- [ ] 📄 I have made corresponding changes to the documentation
- [ ] ⚠️ My changes generate no new warnings
- [ ] 🧪 I have added tests to this description that prove my fix is
effective or that my feature works
- [ ] ⬆️ I increased versions for any altered app according to semantic
versioning

** App addition**

If this PR is an app addition please make sure you have done the
following.

- [ ] 🪞 I have opened a PR on
[truecharts/containers](https://github.com/truecharts/containers) adding
the container to TrueCharts mirror repo.
- [ ] 🖼️ I have added an icon in the Chart's root directory called
`icon.png`

---

_Please don't blindly check all the boxes. Read them and only check
those that apply.
Those checkboxes are there for the reviewer to see what is this all
about and
the status of this PR with a quick glance._
2023-06-04 11:12:44 +02:00
TrueCharts Bot
d629965298 chore(deps): update actions/setup-python digest to bd6b4b6 (#432)
This PR contains the following updates:

| Package | Type | Update | Change |
|---|---|---|---|
| [actions/setup-python](https://togithub.com/actions/setup-python) |
action | digest | `57ded4d` -> `bd6b4b6` |

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined),
Automerge - At any time (no schedule defined).

🚦 **Automerge**: Enabled.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the
rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update
again.

---

- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check
this box

---

This PR has been generated by [Renovate
Bot](https://togithub.com/renovatebot/renovate).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNS4xMDAuMCIsInVwZGF0ZWRJblZlciI6IjM1LjEwMC4wIiwidGFyZ2V0QnJhbmNoIjoibWFpbiJ9-->
common-12.12.1
2023-06-03 16:14:34 +03:00
TrueCharts Bot
6a93e687ba chore(deps): update renovatebot/github-action action to v38 (#435)
This PR contains the following updates:

| Package | Type | Update | Change |
|---|---|---|---|
|
[renovatebot/github-action](https://togithub.com/renovatebot/github-action)
| action | major | `v37.0.0` -> `v38.1.0` |

---

### Release Notes

<details>
<summary>renovatebot/github-action</summary>

###
[`v38.1.0`](https://togithub.com/renovatebot/github-action/releases/tag/v38.1.0)

[Compare
Source](https://togithub.com/renovatebot/github-action/compare/v38.0.0...v38.1.0)

##### Features

- add mount-docker-socket option
([#&#8203;749](https://togithub.com/renovatebot/github-action/issues/749))
([7c65b94](7c65b94442))

##### Documentation

- fix typo in readme "table of contents"
([#&#8203;754](https://togithub.com/renovatebot/github-action/issues/754))
([110eebd](110eebdddc))

##### Continuous Integration

- **deps:** update renovate docker tag to v35.102.0
([da15a1b](da15a1bb63))

###
[`v38.0.0`](https://togithub.com/renovatebot/github-action/releases/tag/v38.0.0)

[Compare
Source](https://togithub.com/renovatebot/github-action/compare/v37.1.0...v38.0.0)

##### ⚠ BREAKING CHANGES

- By default the renovate image is now pulled from
`ghcr.io/renovatebot/renovate`

##### Features

- use ghcr renovate image
([#&#8203;753](https://togithub.com/renovatebot/github-action/issues/753))
([cc52092](cc52092d4f))

###
[`v37.1.0`](https://togithub.com/renovatebot/github-action/releases/tag/v37.1.0)

[Compare
Source](https://togithub.com/renovatebot/github-action/compare/v37.0.0...v37.1.0)

##### Features

- allow override renovate image
([#&#8203;739](https://togithub.com/renovatebot/github-action/issues/739))
([53fd09b](53fd09be22))

##### Documentation

- add github enterprise example
([#&#8203;732](https://togithub.com/renovatebot/github-action/issues/732))
([03f0628](03f0628420))

##### Miscellaneous Chores

- **deps:** update dependency
[@&#8203;tsconfig/node16](https://togithub.com/tsconfig/node16) to
v1.0.4
([a0b6326](a0b63262a9))
- **deps:** update dependency
[@&#8203;types/node](https://togithub.com/types/node) to v16.18.26
([5f87f1b](5f87f1bd59))
- **deps:** update dependency
[@&#8203;types/node](https://togithub.com/types/node) to v16.18.27
([30cf8ba](30cf8ba362))
- **deps:** update dependency
[@&#8203;types/node](https://togithub.com/types/node) to v16.18.28
([36012e3](36012e3e9f))
- **deps:** update dependency
[@&#8203;types/node](https://togithub.com/types/node) to v16.18.29
([4b7dada](4b7dadac33))
- **deps:** update dependency
[@&#8203;types/node](https://togithub.com/types/node) to v16.18.30
([0468c62](0468c62338))
- **deps:** update dependency
[@&#8203;types/node](https://togithub.com/types/node) to v16.18.31
([36edfff](36edfff125))
- **deps:** update dependency eslint to v8.40.0
([441910f](441910f4cd))
- **deps:** update dependency rimraf to v5.0.1
([67ba940](67ba9403a1))
- **deps:** update linters to v5.59.5
([9461668](9461668662))
- **deps:** update linters to v5.59.6
([25a474e](25a474eb3f))
- fix linting
([#&#8203;751](https://togithub.com/renovatebot/github-action/issues/751))
([7e087b0](7e087b03e0))

##### Continuous Integration

- **deps:** update renovate docker tag to v35.100.0
([fd3e2a8](fd3e2a8ccb))
- **deps:** update renovate docker tag to v35.100.3
([#&#8203;752](https://togithub.com/renovatebot/github-action/issues/752))
([bba1483](bba1483352))
- **deps:** update renovate docker tag to v35.80.1
([6e620b5](6e620b5fe9))
- **deps:** update renovate docker tag to v35.81.0
([00fae05](00fae057e3))
- **deps:** update renovate docker tag to v35.82.0
([7dddaac](7dddaac2f3))
- **deps:** update renovate docker tag to v35.87.0
([b1ada6d](b1ada6df90))
- **deps:** update renovate docker tag to v35.89.1
([#&#8203;740](https://togithub.com/renovatebot/github-action/issues/740))
([8006394](80063940ab))
- **deps:** update renovate docker tag to v35.93.0
([#&#8203;741](https://togithub.com/renovatebot/github-action/issues/741))
([277fb1e](277fb1eebc))
- **deps:** update renovate docker tag to v35.94.1
([7d17b82](7d17b82ce8))
- **deps:** update renovate docker tag to v35.95.1
([80896e8](80896e8391))
- **deps:** update renovate docker tag to v35.96.1
([303e01e](303e01e8e8))
- **deps:** update renovate docker tag to v35.96.3
([#&#8203;745](https://togithub.com/renovatebot/github-action/issues/745))
([f32ec46](f32ec46661))
- **deps:** update renovate docker tag to v35.97.0
([d9d5494](d9d5494b9e))
- **deps:** update renovate docker tag to v35.97.1
([13e8e08](13e8e08306))
- **deps:** update renovate docker tag to v35.98.0
([91c814a](91c814a09e))
- **deps:** update renovate docker tag to v35.98.1
([cc7c045](cc7c045174))
- **deps:** update renovate docker tag to v35.98.4
([f53180c](f53180c2c2))
- **deps:** update renovate docker tag to v35.99.0
([#&#8203;748](https://togithub.com/renovatebot/github-action/issues/748))
([a140638](a140638fbb))

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined),
Automerge - At any time (no schedule defined).

🚦 **Automerge**: Enabled.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the
rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update
again.

---

- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check
this box

---

This PR has been generated by [Renovate
Bot](https://togithub.com/renovatebot/renovate).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNS4xMDAuMyIsInVwZGF0ZWRJblZlciI6IjM1LjEwMi4zIiwidGFyZ2V0QnJhbmNoIjoibWFpbiJ9-->
2023-06-03 16:13:54 +03:00
TrueCharts Bot
ac2032287d chore(deps): update container image tccr.io/truecharts/tailscale to v1.42.0 (#445)
This PR contains the following updates:

| Package | Update | Change |
|---|---|---|
|
[tccr.io/truecharts/tailscale](https://togithub.com/truecharts/containers)
| minor | `v1.40.1` -> `v1.42.0` |

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined),
Automerge - At any time (no schedule defined).

🚦 **Automerge**: Enabled.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the
rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update
again.

---

- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check
this box

---

This PR has been generated by [Renovate
Bot](https://togithub.com/renovatebot/renovate).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNS4xMDUuNSIsInVwZGF0ZWRJblZlciI6IjM1LjEwNS41IiwidGFyZ2V0QnJhbmNoIjoibWFpbiJ9-->

---------

Signed-off-by: TrueCharts-Bot <bot@truecharts.org>
Co-authored-by: Stavros Kois <47820033+stavros-k@users.noreply.github.com>
2023-06-03 16:13:25 +03:00
Stavros Kois
c0a4069d53 Fix db password retrievals. (#452)
**Description**
<!--
Please include a summary of the change and which issue is fixed. Please
also include relevant motivation and context. List any dependencies that
are required for this change.

Lookups on the fullname secrets will return empty and will regen new
password each time
-->
⚒️ Fixes  # <!--(issue)-->

Since we have `exapendObjectName: false` we need to lookup in the same
name as the secret is created and not the exapanded name

**⚙️ Type of change**

- [ ] ⚙️ Feature/App addition
- [ ] 🪛 Bugfix
- [ ] ⚠️ Breaking change (fix or feature that would cause existing
functionality to not work as expected)
- [ ] 🔃 Refactor of current code

**🧪 How Has This Been Tested?**
<!--
Please describe the tests that you ran to verify your changes. Provide
instructions so we can reproduce. Please also list any relevant details
for your test configuration
-->

**📃 Notes:**
<!-- Please enter any other relevant information here -->

**✔️ Checklist:**

- [ ] ⚖️ My code follows the style guidelines of this project
- [ ] 👀 I have performed a self-review of my own code
- [ ] #️⃣ I have commented my code, particularly in hard-to-understand
areas
- [ ] 📄 I have made corresponding changes to the documentation
- [ ] ⚠️ My changes generate no new warnings
- [ ] 🧪 I have added tests to this description that prove my fix is
effective or that my feature works
- [ ] ⬆️ I increased versions for any altered app according to semantic
versioning

** App addition**

If this PR is an app addition please make sure you have done the
following.

- [ ] 🪞 I have opened a PR on
[truecharts/containers](https://github.com/truecharts/containers) adding
the container to TrueCharts mirror repo.
- [ ] 🖼️ I have added an icon in the Chart's root directory called
`icon.png`

---

_Please don't blindly check all the boxes. Read them and only check
those that apply.
Those checkboxes are there for the reviewer to see what is this all
about and
the status of this PR with a quick glance._
2023-06-03 16:12:52 +03:00
Kjeld Schouten
368b232d9d Update Chart.yaml common-12.12.0 2023-06-03 13:49:54 +02:00
Kjeld Schouten
59646a80fd Update common.release.yaml 2023-06-03 13:48:43 +02:00
Stavros Kois
71971dc8cd Add chmod to auto perm script (#451)
**Description**
<!--
Please include a summary of the change and which issue is fixed. Please
also include relevant motivation and context. List any dependencies that
are required for this change.
-->
⚒️ Fixes  # <!--(issue)-->

Renamed `permissions` to `ownership`
And added chmod (mentioning `permissions`)

Did some formatting, mostly for syntax highlighting to pop up

**⚙️ Type of change**

- [x] ⚙️ Feature/App addition
- [x] 🪛 Bugfix
- [ ] ⚠️ Breaking change (fix or feature that would cause existing
functionality to not work as expected)
- [x] 🔃 Refactor of current code

**🧪 How Has This Been Tested?**
<!--
Please describe the tests that you ran to verify your changes. Provide
instructions so we can reproduce. Please also list any relevant details
for your test configuration
-->

**📃 Notes:**
<!-- Please enter any other relevant information here -->

**✔️ Checklist:**

- [x] ⚖️ My code follows the style guidelines of this project
- [x] 👀 I have performed a self-review of my own code
- [ ] #️⃣ I have commented my code, particularly in hard-to-understand
areas
- [ ] 📄 I have made corresponding changes to the documentation
- [ ] ⚠️ My changes generate no new warnings
- [ ] 🧪 I have added tests to this description that prove my fix is
effective or that my feature works
- [x] ⬆️ I increased versions for any altered app according to semantic
versioning

** App addition**

If this PR is an app addition please make sure you have done the
following.

- [ ] 🪞 I have opened a PR on
[truecharts/containers](https://github.com/truecharts/containers) adding
the container to TrueCharts mirror repo.
- [ ] 🖼️ I have added an icon in the Chart's root directory called
`icon.png`

---

_Please don't blindly check all the boxes. Read them and only check
those that apply.
Those checkboxes are there for the reviewer to see what is this all
about and
the status of this PR with a quick glance._

---------

Co-authored-by: Kjeld Schouten <kjeld@schouten-lebbing.nl>
2023-06-03 13:37:17 +02:00
Kjeld Schouten
abc36d3bb8 Label fix (#450)
**Description**
Working on our CNPG chart, I discovered some of the basic adviced
kubernetes labels are put into selectorLabels.

This is odd behavior, because they should be put everywhere. For CNPG
this lead to it not being able to find deployments as those where
lacking basic selector labels and I expect this to happen for other
objects as well in the future.

However, as this include is also used to match things, we need to keep
it in both includes.
As they are the same, duplicates should just be merged out without
issue.

**⚙️ Type of change**

- [ ] ⚙️ Feature/App addition
- [x] 🪛 Bugfix
- [ ] ⚠️ Breaking change (fix or feature that would cause existing
functionality to not work as expected)
- [ ] 🔃 Refactor of current code

**🧪 How Has This Been Tested?**
<!--
Please describe the tests that you ran to verify your changes. Provide
instructions so we can reproduce. Please also list any relevant details
for your test configuration
-->

**📃 Notes:**
<!-- Please enter any other relevant information here -->

**✔️ Checklist:**

- [ ] ⚖️ My code follows the style guidelines of this project
- [ ] 👀 I have performed a self-review of my own code
- [ ] #️⃣ I have commented my code, particularly in hard-to-understand
areas
- [ ] 📄 I have made corresponding changes to the documentation
- [ ] ⚠️ My changes generate no new warnings
- [ ] 🧪 I have added tests to this description that prove my fix is
effective or that my feature works
- [ ] ⬆️ I increased versions for any altered app according to semantic
versioning

** App addition**

If this PR is an app addition please make sure you have done the
following.

- [ ] 🪞 I have opened a PR on
[truecharts/containers](https://github.com/truecharts/containers) adding
the container to TrueCharts mirror repo.
- [ ] 🖼️ I have added an icon in the Chart's root directory called
`icon.png`

---

_Please don't blindly check all the boxes. Read them and only check
those that apply.
Those checkboxes are there for the reviewer to see what is this all
about and
the status of this PR with a quick glance._

---------

Co-authored-by: Stavros kois <s.kois@outlook.com>
common-12.11.1
2023-06-03 00:07:57 +02:00
Kjeld Schouten
18fab9e788 Update common_library_tests.yaml 2023-06-03 00:07:36 +02:00
Kjeld Schouten
8553aa114e Update common_library_tests.yaml 2023-06-03 00:02:16 +02:00
Kjeld Schouten
78cc9d5438 Delete metadata-label-commenter.yaml 2023-06-02 23:59:19 +02:00
Kjeld Schouten
bcaa9642a1 Delete metadata-label-pr.yaml 2023-06-02 23:58:57 +02:00
Kjeld Schouten
47878e4329 feat(common): add support for additional templates defined in values.yaml (#448)
**Description**
add extraTpl support for custom template file defining in values.yaml

**⚙️ Type of change**

- [ ] ⚙️ Feature/App addition
- [ ] 🪛 Bugfix
- [ ] ⚠️ Breaking change (fix or feature that would cause existing
functionality to not work as expected)
- [ ] 🔃 Refactor of current code

**🧪 How Has This Been Tested?**
<!--
Please describe the tests that you ran to verify your changes. Provide
instructions so we can reproduce. Please also list any relevant details
for your test configuration
-->

**📃 Notes:**
<!-- Please enter any other relevant information here -->

**✔️ Checklist:**

- [ ] ⚖️ My code follows the style guidelines of this project
- [ ] 👀 I have performed a self-review of my own code
- [ ] #️⃣ I have commented my code, particularly in hard-to-understand
areas
- [ ] 📄 I have made corresponding changes to the documentation
- [ ] ⚠️ My changes generate no new warnings
- [ ] 🧪 I have added tests to this description that prove my fix is
effective or that my feature works
- [ ] ⬆️ I increased versions for any altered app according to semantic
versioning

** App addition**

If this PR is an app addition please make sure you have done the
following.

- [ ] 🪞 I have opened a PR on
[truecharts/containers](https://github.com/truecharts/containers) adding
the container to TrueCharts mirror repo.
- [ ] 🖼️ I have added an icon in the Chart's root directory called
`icon.png`

---

_Please don't blindly check all the boxes. Read them and only check
those that apply.
Those checkboxes are there for the reviewer to see what is this all
about and
the status of this PR with a quick glance._

---------

Co-authored-by: Stavros kois <s.kois@outlook.com>
2023-06-02 23:37:31 +02:00
Kjeld Schouten
aae239e954 fix(common): default to tcp startup probe for multi-stage probes (#449)
**Description**
This splits the default probe design to define the default container as
"running" when it responds on it's port and "ready to accept
connections" when it actually has a valid http code on said port.

it's a design frequently employed by some other big helm chart creators
and put the seperate startup-probe to good use.
It also allows us to check if it's a http or port/application issue when
running charts through tests. As those would fail on different probes.

**⚙️ Type of change**

- [x] ⚙️ Feature/App addition
- [ ] 🪛 Bugfix
- [ ] ⚠️ Breaking change (fix or feature that would cause existing
functionality to not work as expected)
- [x] 🔃 Refactor of current code

**🧪 How Has This Been Tested?**
<!--
Please describe the tests that you ran to verify your changes. Provide
instructions so we can reproduce. Please also list any relevant details
for your test configuration
-->

**📃 Notes:**
<!-- Please enter any other relevant information here -->

**✔️ Checklist:**

- [ ] ⚖️ My code follows the style guidelines of this project
- [ ] 👀 I have performed a self-review of my own code
- [ ] #️⃣ I have commented my code, particularly in hard-to-understand
areas
- [ ] 📄 I have made corresponding changes to the documentation
- [ ] ⚠️ My changes generate no new warnings
- [ ] 🧪 I have added tests to this description that prove my fix is
effective or that my feature works
- [ ] ⬆️ I increased versions for any altered app according to semantic
versioning

** App addition**

If this PR is an app addition please make sure you have done the
following.

- [ ] 🪞 I have opened a PR on
[truecharts/containers](https://github.com/truecharts/containers) adding
the container to TrueCharts mirror repo.
- [ ] 🖼️ I have added an icon in the Chart's root directory called
`icon.png`

---

_Please don't blindly check all the boxes. Read them and only check
those that apply.
Those checkboxes are there for the reviewer to see what is this all
about and
the status of this PR with a quick glance._
common-12.10.14
2023-06-02 14:56:36 +02:00
Stavros Kois
15db491b14 chore(tests); add tests for optional (#447)
**Description**
<!--
Please include a summary of the change and which issue is fixed. Please
also include relevant motivation and context. List any dependencies that
are required for this change.
-->
⚒️ Fixes  # <!--(issue)-->

**⚙️ Type of change**

- [ ] ⚙️ Feature/App addition
- [ ] 🪛 Bugfix
- [ ] ⚠️ Breaking change (fix or feature that would cause existing
functionality to not work as expected)
- [ ] 🔃 Refactor of current code

**🧪 How Has This Been Tested?**
<!--
Please describe the tests that you ran to verify your changes. Provide
instructions so we can reproduce. Please also list any relevant details
for your test configuration
-->

**📃 Notes:**
<!-- Please enter any other relevant information here -->

**✔️ Checklist:**

- [ ] ⚖️ My code follows the style guidelines of this project
- [ ] 👀 I have performed a self-review of my own code
- [ ] #️⃣ I have commented my code, particularly in hard-to-understand
areas
- [ ] 📄 I have made corresponding changes to the documentation
- [ ] ⚠️ My changes generate no new warnings
- [ ] 🧪 I have added tests to this description that prove my fix is
effective or that my feature works
- [ ] ⬆️ I increased versions for any altered app according to semantic
versioning

** App addition**

If this PR is an app addition please make sure you have done the
following.

- [ ] 🪞 I have opened a PR on
[truecharts/containers](https://github.com/truecharts/containers) adding
the container to TrueCharts mirror repo.
- [ ] 🖼️ I have added an icon in the Chart's root directory called
`icon.png`

---

_Please don't blindly check all the boxes. Read them and only check
those that apply.
Those checkboxes are there for the reviewer to see what is this all
about and
the status of this PR with a quick glance._
2023-06-02 14:49:47 +02:00
Kjeld Schouten-Lebbing
459503cd48 Revert "fix: move to tcp startup-probe"
This reverts commit a4e3178167.
2023-06-02 14:28:14 +02:00
Kjeld Schouten
a4e3178167 fix: move to tcp startup-probe 2023-06-02 14:27:05 +02:00