Skip to content

Fix quota.yaml permission issue#430

Merged
jose-caballero merged 1 commit into
mainfrom
dev-0pz-patch-1
Jun 22, 2026
Merged

Fix quota.yaml permission issue#430
jose-caballero merged 1 commit into
mainfrom
dev-0pz-patch-1

Conversation

@dev-0pz

@dev-0pz dev-0pz commented Mar 17, 2026

Copy link
Copy Markdown
Contributor

Description:

POST 
 400 (Bad Request)
 
Unable to schedule action "stackstorm_openstack.quota.show".
 
Error: Request failed with status code 400
    at createError (
)
    at settle (
)
    at XMLHttpRequest.onloadend (
)

Quota.yaml doesnt allow for me to be able to use quota show since enable is set to false

Special Notes:


Submitter:

Have you (where applicable):

  • Added unit tests?
  • Checked the latest commit runs on Dev?
  • Updated the example config file(s) and README?

Reviewer

Does this PR:

  • Place non-StackStorm code into the lib directory?
  • Have unit tests for the action/sensor and lib layers?
  • Have clear and obvious action parameter names and descriptions?

@codecov

codecov Bot commented Mar 17, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 99.81%. Comparing base (3ffaa4f) to head (2c3bff8).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #430   +/-   ##
=======================================
  Coverage   99.81%   99.81%           
=======================================
  Files         109      109           
  Lines        2643     2643           
  Branches      331      331           
=======================================
  Hits         2638     2638           
  Misses          4        4           
  Partials        1        1           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@jose-caballero jose-caballero left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Note that the actual function may not do anything, as it doesn't return anything. It may just unconditionally print out the values. But can only check it by trying it, which requires the action to be enable :)

@jose-caballero jose-caballero merged commit 9315891 into main Jun 22, 2026
14 checks passed
@jose-caballero jose-caballero deleted the dev-0pz-patch-1 branch June 22, 2026 08:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants