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

[observium] Pod Security Context - Expected [enabled] in [persistence.observium] to exist #19193

Closed
1 task done
bass000 opened this issue Mar 14, 2024 · 4 comments · Fixed by #19196
Closed
1 task done
Assignees
Labels
bug Something isn't working

Comments

@bass000
Copy link

bass000 commented Mar 14, 2024

App Name

observium

Operating System

TrueNAS SCALE 23.10.1

App Version

7.1.3

Application Events

app not installed

Application Logs

app not installed

Application Configuration

app not installed

Describe the bug

Install. All default (apart from username/password and ingress info)
Failed to install App: Pod Security Context - Expected the key [enabled] in [persistence.observium] to exist
remove_circle_outline
More info...
Error: Traceback (most recent call last):
File "/usr/lib/python3/dist-packages/middlewared/job.py", line 426, in run
await self.future
File "/usr/lib/python3/dist-packages/middlewared/job.py", line 464, in __run_body
rv = await self.method(([self] + args))
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/lib/python3/dist-packages/middlewared/service/crud_service.py", line 194, in nf
rv = await func(args, kwargs)
^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/lib/python3/dist-packages/middlewared/schema/processor.py", line 44, in nf
res = await f(*args, kwargs)
^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/lib/python3/dist-packages/middlewared/schema/processor.py", line 177, in nf
return await func(args, **kwargs)
^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/lib/python3/dist-packages/middlewared/plugins/chart_releases_linux/chart_release.py", line 487, in do_create
await self.middleware.call(
File "/usr/lib/python3/dist-packages/middlewared/main.py", line 1398, in call
return await self._call(
^^^^^^^^^^^^^^^^^
File "/usr/lib/python3/dist-packages/middlewared/main.py", line 1352, in _call
return await self.run_in_executor(prepared_call.executor, methodobj,prepared_call.args)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/lib/python3/dist-packages/middlewared/main.py", line 1251, in run_in_executor
return await loop.run_in_executor(pool, functools.partial(method, *args, **kwargs))

To Reproduce

install by default

Expected Behavior

install works

Screenshots

not relevant

Additional Context

not relevant

I've read and agree with the following

  • I've checked all open and closed issues and my issue is not there.
@bass000 bass000 added the bug Something isn't working label Mar 14, 2024
@sethidden

This comment was marked as spam.

@Ornias1993
Copy link
Member

@sethidden
Don't spam unrelated issues, your comments will only get hidden and ignored.

File your own issues for seperate apps/

@truecharts truecharts locked as spam and limited conversation to collaborators Mar 14, 2024
@Ornias1993
Copy link
Member

@bass000 Thanks for your submission

These issues now become visible because some app-submitters made mistakes having a desynced questions.yaml and values.yaml and/or added objects without specifying weither they should be enabled.

We've since this week added extra checks to prevent charts to run if thats the case, as it might/will lead to unexpected behavior.

@Ornias1993 Ornias1993 changed the title Failed to install App: Pod Security Context - Expected the key [enabled] in [persistence.observium] to exist [observium] Pod Security Context - Expected [enabled] in [persistence.observium] to exist Mar 14, 2024
@Ornias1993
Copy link
Member

please next time stick with our mandatory title format though

xstar97 added a commit that referenced this issue Mar 14, 2024
**Description**

Fixes the persistence mismatch between values and questions

⚒️ Fixes  #19193 

**⚙️ 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 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: Xstar97TheNoob <9399967+xstar97@users.noreply.github.com>
Co-authored-by: Xstar97TheNoob <9399967+xstar97@users.noreply.github.com>
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
4 participants