Commit Graph

37 Commits

Author SHA1 Message Date
Stavros Kois 428d5218d8
fix(ci): handle better fetch deps (#15792)
**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-12-06 13:28:53 +02:00
Kjeld Schouten 0e84404e32
Update fetch_helm_deps.sh
Signed-off-by: Kjeld Schouten <kjeld@schouten-lebbing.nl>
2023-12-05 19:57:48 +01:00
Kjeld Schouten d1c4cf28fd
Update fetch_helm_deps.sh
Signed-off-by: Kjeld Schouten <kjeld@schouten-lebbing.nl>
2023-12-05 19:47:58 +01:00
Kjeld Schouten e743e7f8ec
Update fetch_helm_deps.sh
Signed-off-by: Kjeld Schouten <kjeld@schouten-lebbing.nl>
2023-12-05 19:39:08 +01:00
Kjeld Schouten 27664e7ff3
feat(metallb/cloudnative-pg/CI): BREAKING CHANGE move to consume upstream helm charts (#14681)
**Description**

To ensure stability, we've decided to directly use upstream helm-charts
from their respective projects for operators specifically.
Mostly because they heavily rely on rbac, crds and crs, which could
cause conflicts between shipped rbac/crd and container versions.

also patches the ci for a few other operators to exclude them from
signing requirements

⚒️ Fixes  # <!--(issue)-->

**⚙️ Type of change**

- [ ] ⚙️ Feature/App addition
- [ ] 🪛 Bugfix
- [x] ⚠️ 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.

- [x] 🪞 I have opened a PR on
[truecharts/containers](https://github.com/truecharts/containers) adding
the container to TrueCharts mirror repo.
- [x] 🖼️ 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-11-15 15:39:03 +01:00
Kjeld Schouten dc55c9a77d
feat(velero): add velero (#14648)
**Description**
This adds a Velero helm chart.
This time we've chosen to directly use the upstream operator helm chart,
as it does not contain many problematic bits and would offer more
stability.

⚒️ Fixes  #14524

**⚙️ Type of change**

- [x] ⚙️ 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:**
This does not contain any custom resources references backed by common
yet.
However, those can also be added using velero cli for testing for the
time being.

It also does not contain any GUI sections for SCALE yet, just like
cert-manager it would likely be added on-demand.

**✔️ 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.

- [x] 🪞 I have opened a PR on
[truecharts/containers](https://github.com/truecharts/containers) adding
the container to TrueCharts mirror repo.
- [x] 🖼️ 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-11-14 14:12:27 +01:00
Kjeld Schouten 3fb1c6a2ea
feat: add cert-manager operator chart DNM DRAFT (#10297)
**Description**
Draft addition of cert-manager as an operator chart

**⚙️ 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-07-15 19:22:49 +02:00
Stavros Kois d5cbee2d65
pure regex
Signed-off-by: Stavros Kois <47820033+stavros-k@users.noreply.github.com>
2023-05-25 20:45:41 +03:00
Stavros Kois d4276bdbea
uhhh
Signed-off-by: Stavros Kois <47820033+stavros-k@users.noreply.github.com>
2023-05-25 20:44:32 +03:00
Stavros Kois f79684a8ca
Update fetch_helm_deps.sh
Signed-off-by: Stavros Kois <47820033+stavros-k@users.noreply.github.com>
2023-05-25 20:32:44 +03:00
Stavros Kois 3eb1d28fbe
fix condition. It's train/chart
Signed-off-by: Stavros Kois <47820033+stavros-k@users.noreply.github.com>
2023-05-25 20:22:23 +03:00
Kjeld Schouten 2b21fe0c21
Update fetch_helm_deps.sh
Signed-off-by: Kjeld Schouten <kjeld@schouten-lebbing.nl>
2023-05-25 13:40:54 +02:00
Kjeld Schouten 6edf7ab0ca
Update fetch_helm_deps.sh
Signed-off-by: Kjeld Schouten <kjeld@schouten-lebbing.nl>
2023-05-25 13:39:13 +02:00
Kjeld Schouten d65db11922
Update fetch_helm_deps.sh
Signed-off-by: Kjeld Schouten <kjeld@schouten-lebbing.nl>
2023-05-25 12:07:45 +02:00
Kjeld Schouten 040e87f8ec
Update fetch_helm_deps.sh
Signed-off-by: Kjeld Schouten <kjeld@schouten-lebbing.nl>
2023-05-24 20:50:31 +02:00
Kjeld Schouten 10f782584e
Update fetch_helm_deps.sh
Signed-off-by: Kjeld Schouten <kjeld@schouten-lebbing.nl>
2023-05-24 20:22:33 +02:00
Kjeld Schouten-Lebbing 0fcf0da235 chore: add some patches to prepare for operators train 2023-05-23 14:14:16 +02:00
Stavros Kois 702a0c3aba
chore(ci): Improved lint speed (#8809)
**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)-->

TODO: Switch to scripted checks (To replace ct-list's tasks):
- [x] Helm Lint
- [x] Version Checking
- [x] Run the lint inside the devcontainer (so it have the below tools
available)
- [x] YAML Schema check on Chart.yaml
- [ ] Maintener check on Chart.yaml (Moved to another PR in the future)
- [x] YAML Lint on Chart.yaml
- [x] YAML Lint on values.yaml
- [x] Sort output of lint results (failed on the bottom)
- [x] Combine verify deps in the lint job, since the verification runs
inside the fetch dep (which runs on lint job too)
- [x] Remove one of the 2 "get changed" functions (one based on tags
other on commit)
After that, move linting to single step 


https://github.com/helm/chart-testing/blob/main/pkg/tool/linter.go
https://github.com/helm/chart-testing/blob/main/pkg/tool/account.go

**⚙️ 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
- [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._
2023-05-23 10:26:41 +02:00
Kjeld Schouten 380156b8ca
Update fetch_helm_deps.sh
Signed-off-by: Kjeld Schouten <kjeld@schouten-lebbing.nl>
2023-05-07 13:39:56 +02:00
Kjeld Schouten 48c96504c3
Update fetch_helm_deps.sh
Signed-off-by: Kjeld Schouten <kjeld@schouten-lebbing.nl>
2023-05-07 13:32:49 +02:00
Kjeld Schouten 5120931092
Update fetch_helm_deps.sh
Signed-off-by: Kjeld Schouten <kjeld@schouten-lebbing.nl>
2023-05-07 13:14:59 +02:00
Kjeld Schouten 29a8c18c1f
Update fetch_helm_deps.sh
Signed-off-by: Kjeld Schouten <kjeld@schouten-lebbing.nl>
2023-05-07 12:48:52 +02:00
Kjeld Schouten b6b8387a0c
Update ct-lint.yaml (#8789)
**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._

---------

Signed-off-by: Kjeld Schouten <kjeld@schouten-lebbing.nl>
2023-05-05 22:36:14 +02:00
Kjeld Schouten 6a7c125052
feat(ci): add validation of dependency chart signatures
Signed-off-by: Kjeld Schouten <kjeld@schouten-lebbing.nl>
2023-05-05 20:05:46 +02:00
Stavros Kois 881c939dc0
feat(ci): Download each dep to each own repo dir, and use a cached index instead of fetch each time (#4393)
* download each dep to its repo dir, to avoid confclits

* only fetch index once

* clean index cache on start

* fancy print

* make sure it's always `index.yaml`

* add caches to gitignore

* empty lines

* don't wipe
2022-11-12 19:06:22 +01:00
Stavros Kois 86c3ca7a7b
fix(ci): fix a check (#4388)
* chore(ci): fix a visual typo

* fix copy..

* its train/chart :D

* Add some comments so I can read it again in the feature :P

* comment

* fix a check..
2022-11-12 17:09:02 +01:00
Stavros Kois f7ddd8f555
Add some comments to the fetch deps script.
Signed-off-by: Stavros Kois <47820033+stavros-k@users.noreply.github.com>
2022-11-12 18:02:16 +02:00
Stavros Kois dcc7fe5ed3
chore(ci): fix cp path (#4387)
* chore(ci): fix a visual typo

* fix copy..

* its train/chart :D
2022-11-12 16:58:44 +01:00
Kjeld Schouten-Lebbing 37ea4a1920
Update fetch_helm_deps.sh
Signed-off-by: Kjeld Schouten-Lebbing <kjeld@schouten-lebbing.nl>
2022-11-12 16:45:32 +01:00
Kjeld Schouten-Lebbing 318fde858e
Update fetch_helm_deps.sh
Signed-off-by: Kjeld Schouten-Lebbing <kjeld@schouten-lebbing.nl>
2022-11-12 16:32:22 +01:00
Kjeld Schouten-Lebbing 9c774c720f
Update fetch_helm_deps.sh
Signed-off-by: Kjeld Schouten-Lebbing <kjeld@schouten-lebbing.nl>
2022-11-12 16:27:53 +01:00
Kjeld Schouten-Lebbing 9f38ddbf60
Update fetch_helm_deps.sh
Signed-off-by: Kjeld Schouten-Lebbing <kjeld@schouten-lebbing.nl>
2022-11-12 16:22:44 +01:00
Kjeld Schouten-Lebbing 07c6d1ab7e
Update fetch_helm_deps.sh
Signed-off-by: Kjeld Schouten-Lebbing <kjeld@schouten-lebbing.nl>
2022-11-12 16:00:47 +01:00
Kjeld Schouten-Lebbing 8c25487408
Update fetch_helm_deps.sh
Signed-off-by: Kjeld Schouten-Lebbing <kjeld@schouten-lebbing.nl>
2022-11-12 15:37:44 +01:00
Kjeld Schouten-Lebbing 27548259e2
Update fetch_helm_deps.sh
Signed-off-by: Kjeld Schouten-Lebbing <kjeld@schouten-lebbing.nl>
2022-11-12 15:33:44 +01:00
Kjeld Schouten-Lebbing acb92de675
Update fetch_helm_deps.sh
Signed-off-by: Kjeld Schouten-Lebbing <kjeld@schouten-lebbing.nl>
2022-11-12 15:33:27 +01:00
Stavros Kois 4d335f300d
feat(ci): add fetch helm deps script (#4386)
* feat(ci): add fetch helm deps

* add some error handling

* require the command

* pretty output
2022-11-12 15:27:40 +01:00