Commit Graph

21 Commits

Author SHA1 Message Date
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