Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Splunk chart uses non-existent middleware #15091

Closed
2 tasks done
atanaspam opened this issue Nov 18, 2023 · 2 comments · Fixed by #15093
Closed
2 tasks done

Splunk chart uses non-existent middleware #15091

atanaspam opened this issue Nov 18, 2023 · 2 comments · Fixed by #15093
Labels
bug Something isn't working

Comments

@atanaspam
Copy link
Contributor

atanaspam commented Nov 18, 2023

App Name

Splunk

Operating System

TrueNAS SCALE 22.12.4.2

App Version

9.0.4_3.0.10

Application Events

2023-11-18 23:24:47
Job completed
2023-11-18 23:24:42
Created container splunk-manifests
2023-11-18 23:24:42
Started container splunk-manifests
2023-11-18 23:24:41
Add eth0 [172.16.0.159/16] from ix-net
2023-11-18 23:24:41
Container image "tccr.io/truecharts/kubectl:v1.26.0@sha256:6d6e0e50f28b961ed1c1c6a9c140553238641591fbdc9ac7c1a348636f78c552" already present on machine
2023-11-18 23:24:40
Created pod: splunk-manifests-hvxsq
2023-11-18 23:24:40
Successfully assigned ix-splunk/splunk-manifests-hvxsq to ix-truenas

Application Logs

N/A

Application Configuration

No changes to Networking and Services

Ingress

  • Enable Ingress

Hosts

HostName*
splunk.internal.mydomain.com
Paths

Path*
/
Path Type*
Prefix
clusterIssuer
letsencrypt-staging
(Advanced) Traefik Entrypoint*
webecure

No further changes to the default config

Describe the bug

After the app is successfully deployed and running, the web interface is still unavailable due to a misconfiguration in traefik. The traefik dashboard reports the following error for the Splunk router.

middleware "default-chain-basic@kubernetescrd" does not exist 

I can see that the middleware mentioned above is indeed set in the ingress annotations:

root@truenas[~]# k3s kubectl describe ingress splunk -n ix-splunk                 
Name:             splunk
Labels:           app.kubernetes.io/instance=splunk
                  app.kubernetes.io/managed-by=Helm
                  app.kubernetes.io/name=splunk
                  app.kubernetes.io/version=9.0.4
                  helm-revision=12
                  helm.sh/chart=splunk-3.0.10
Namespace:        ix-splunk
Address:          192.168.1.10
Ingress Class:    <none>
Default backend:  <default>
Rules:
  Host                        Path  Backends
  ----                        ----  --------
  splunk.internal.mydomain.com
                              /   splunk:8000 (172.16.0.150:8000)
Annotations:                  meta.helm.sh/release-name: splunk
                              meta.helm.sh/release-namespace: ix-splunk
                              traefik.ingress.kubernetes.io/router.entrypoints: websecure
                              traefik.ingress.kubernetes.io/router.middlewares: default-chain-basic@kubernetescrd
Events:                       <none>

However, I can see that a middleware with the name: ix-traefik-chain-basic does exist in traefik.

I suspect this is caused by the outdated version of the library-charts for the Splunk Chart. The version used is 11.1.2 and I can see that default is indeed is being the default middleware prefix for the library charts versioned 11.1.2:
https://github.com/truecharts/library-charts/blob/0898c5dee7b0ff9bb11ab2b4c8c2870cce61a697/charts/common/templates/class/_ingress.tpl#L37

If simply bumping the library version is enough, I am happy to raise a PR.

To Reproduce

  1. Install latest version of Splunk app from incubating train
  2. Configure ingress with values mentioned in Application Configuration
  3. Inspect traefik dashboard

Expected Behavior

Splunk service is reachable via traefik and the specified hostname.

Screenshots

N/A

Additional Context

N/A

I've read and agree with the following

  • I've checked all open and closed issues and my issue is not there.
@atanaspam atanaspam added the bug Something isn't working label Nov 18, 2023
@PrivatePuffin
Copy link
Member

Incubator means "alpha" "beta" or "unstable", it might not work at all or be heavily outdated.
Hence we don't take bugreports for them either,

@PrivatePuffin PrivatePuffin closed this as not planned Won't fix, can't repro, duplicate, stale Nov 19, 2023
xstar97 added a commit that referenced this issue Nov 23, 2023
**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  #15091

**⚙️ 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
- [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: Atanas Pamukchiev <atanaspam@users.noreply.github.com>
Co-authored-by: Xstar97TheNoob <9399967+xstar97@users.noreply.github.com>
Co-authored-by: StevenMcElligott <89483932+StevenMcElligott@users.noreply.github.com>
@truecharts-admin
Copy link
Collaborator

This issue is locked to prevent necro-posting on closed issues. Please create a new issue or contact staff on discord of the problem persists

@truecharts truecharts locked and limited conversation to collaborators Nov 3, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants