Commit Graph

14 Commits

Author SHA1 Message Date
Kjeld Schouten 4df015c950
Update tc-chains.yaml
Signed-off-by: Kjeld Schouten <kjeld@schouten-lebbing.nl>
2023-05-24 19:00:35 +02:00
Kjeld Schouten-Lebbing 023e095d78 fix(traefik): ensure ingressclass state is fetched from global scope 2023-05-24 11:36:22 +02:00
Kjeld Schouten a2a7c2a305
refactor(traefik): BREAKING CHANGE move traefik to new CRD's and move config to tc-system (#9116)
**Description**
Traefik couldn't be updated due to CRD namespace chagnes.
This PR changes the CRD namespace to `traefik.io` as required, so we can
actually update traefik.

Besides this, it moves the portalhook from either `default` or
`tc-*ingressclassname*`
to `tc-system` (our project config storage namespace). In case of
ingressclass use, `portalhook` will get a suffix instead

the reason behind this is, is the fact non-SCALE users currently cannot
use ingressclass at all and neither can platforms without a `default`
namespace.

This will require all apps to be updated accordingly, as the name and
location of the middleware also changes to `tc-system`, including a
suffix with the ingressClass name where needed.

This move of portalhook, also allows us to fix the "loadbalancer port
suffixed to ingress" bug on SCALE portal button.

**⚙️ 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:**
To be clear: IngressClass is still not 100% supported.
But that doesn't mean we should make it inherently problematic.

The "Breaking" portion of this PR, is just the fact all charts/apps have
to be updated accordingly or otherwise would not function with ingress
anymore. It does not require manual intervention byond updating, hoever.

**✔️ 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-05-24 09:54:40 +02:00
Jean-François Roy 6e2002d7fb
fix(traefik) Fix arguments to fix metrics (#7827)
**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)-->

The _args template did not handle the standard metrics question and thus
the `--metrics.prometheus.entrypoint=metrics` was not passed to traefik,
resulting in 404 errors in prometheus.

**⚙️ 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
-->

Run `helm template --namespace=ix-traefik --set
metrics.main.enabled=true .` in the charts directory. Check that `args`
for the traefik pods contains

- "--entryPoints.metrics.address=:9180/tcp"
- "--metrics.prometheus=true"
- "--metrics.prometheus.entrypoint=metrics"

**📃 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._

---------

Signed-off-by: Kjeld Schouten-Lebbing <kjeld@schouten-lebbing.nl>
Co-authored-by: Kjeld Schouten-Lebbing <kjeld@schouten-lebbing.nl>
2023-03-25 10:27:00 +01:00
Stavros Kois 46c15aed02
fix(traefik): fix portal button (#7811)
**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-03-19 12:58:22 +02:00
Kjeld Schouten-Lebbing 4a625398f5
Update _args.tpl
Signed-off-by: Kjeld Schouten-Lebbing <kjeld@schouten-lebbing.nl>
2023-03-11 21:25:12 +01:00
Stavros Kois 7b37008571
fix(traefik): actually fix labels (#7763) 2023-03-07 22:00:35 +02:00
Stavros Kois d3c8d46763
fix(traefik): fix labels (#7759) 2023-03-07 10:17:26 +01:00
Kjeld Schouten-Lebbing c34bfb1e9b
BREAKING CHANGE refactor: port all enterprise apps to new common and add apps (#7738)
* use new common

* more porting and add notes.txt

* portals

* add vaultwarden and authelia to enterprise

* some changes

* authelia porting

* fix grafana

* fixup metallb

* more

* traefik

* some initial blocky work

* fixes

* more work on enterprise train

* containers

* labels

* no message

* some more fixes

* update questions for new enterprise apps

* something

* remove postgresql dependency from enterprise train apps

* fix some traefik bugs

* remove prometheus affinities for now

* authelia postgresql fixes

* bump and fix install test errors

* bump common for probe fixes

* fix questions

* more questions fixes

* add some metrics improvements

* some more fixes

* whoops

* some authelia fixes

* fix blocky and authelia

* bump common for postgresql fixes

* hmm

* bump common

* bump redis to disable double manifest loading

* dont enc secrets for authelia

* traefik, blocky and authelia fixes

* traefik caps on protocols

* bump redis for password fixes

* ensure roles are clusterwide

* ok

* redis/common bumps

* remove blocky webui and change config location

* whoops

* redis fix

* more blocky tryouts

* authelia whoops

* bump blocky version

* disable prometheus controller

* oops

* bump common on prometheus for custom service selector labels

* rename prometheus selector labels on service

* damn

* more work

* blocky 100 tryout

* blocky root tryout

* fix blocky config and remove fsgroup

* dont drop caps

* Update common.yaml

Signed-off-by: Kjeld Schouten-Lebbing <kjeld@schouten-lebbing.nl>

* Update values.yaml

Signed-off-by: Kjeld Schouten-Lebbing <kjeld@schouten-lebbing.nl>

* Update values.yaml

Signed-off-by: Kjeld Schouten-Lebbing <kjeld@schouten-lebbing.nl>

* revert update for blocky

* use old-style mount for blocky

* put update back for blocky

* add initial postgresl query log support

* hmm

* small lint

* bump common

---------

Signed-off-by: Kjeld Schouten-Lebbing <kjeld@schouten-lebbing.nl>
Co-authored-by: Stavros kois <s.kois@outlook.com>
Co-authored-by: Stavros Kois <47820033+stavros-k@users.noreply.github.com>
2023-03-04 13:42:14 +01:00
Stavros Kois 1ecd4fc869
chore(traefik): Add blacklistMode on geoblock and fix label (#6416) 2023-01-16 11:06:03 +02:00
Kjeld Schouten-Lebbing 8652a8f3df
feat: move Grafana, Prometheus and Traefik to Enterprise Train (#6372) 2023-01-14 15:09:50 +00:00
Kjeld Schouten-Lebbing 433b148de2
chore(enterprise): remove most enterprise apps for now 2022-09-24 23:55:59 +02:00
Kjeld Schouten-Lebbing a93151fb33
chore(enterprise): bump major on new enterprise app to seperate helm updates 2022-09-24 23:32:55 +02:00
Kjeld Schouten-Lebbing edf3f9719f
chore(enterprise): Add a copy of Initial Apps selected for Enterprise Train (#3749)
* chore(enterprise): Add a copy of Initial Apps selected for Enterprise Train

* removing authelia and freeradius for the time being due to depending on openldap and such

* Create README.md

Signed-off-by: Kjeld Schouten-Lebbing <kjeld@schouten-lebbing.nl>

* Create README.md

Signed-off-by: Kjeld Schouten-Lebbing <kjeld@schouten-lebbing.nl>

* Create README.md

Signed-off-by: Kjeld Schouten-Lebbing <kjeld@schouten-lebbing.nl>

* Create README.md

Signed-off-by: Kjeld Schouten-Lebbing <kjeld@schouten-lebbing.nl>

* Create README.md

Signed-off-by: Kjeld Schouten-Lebbing <kjeld@schouten-lebbing.nl>

Signed-off-by: Kjeld Schouten-Lebbing <kjeld@schouten-lebbing.nl>
2022-09-24 21:09:58 +02:00