Skip to content

Conversation

@jremy42
Copy link
Contributor

@jremy42 jremy42 commented Dec 4, 2025

close #3332

@jremy42 jremy42 force-pushed the feat/cockpit-actions branch from 5583d9b to 7232c31 Compare December 4, 2025 10:08
@codecov-commenter
Copy link

codecov-commenter commented Dec 4, 2025

Codecov Report

❌ Patch coverage is 0% with 86 lines in your changes missing coverage. Please review.
✅ Project coverage is 1.76%. Comparing base (c9cdc2d) to head (417444e).
✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
...rvices/cockpit/action_trigger_test_alert_action.go 0.00% 73 Missing ⚠️
internal/locality/regional/schemas_framework.go 0.00% 12 Missing ⚠️
provider/framework.go 0.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff            @@
##           master   #3499      +/-   ##
=========================================
- Coverage    1.76%   1.76%   -0.01%     
=========================================
  Files         406     408       +2     
  Lines       44434   44520      +86     
=========================================
- Hits          786     784       -2     
- Misses      43561   43648      +87     
- Partials       87      88       +1     

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

@jremy42 jremy42 force-pushed the feat/cockpit-actions branch 2 times, most recently from f0e4b54 to 16c2991 Compare December 4, 2025 10:40
@jremy42 jremy42 marked this pull request as ready for review December 5, 2025 08:20
@jremy42 jremy42 requested review from a team and remyleone as code owners December 5, 2025 08:20
@jremy42 jremy42 force-pushed the feat/cockpit-actions branch from 83bef8a to 417444e Compare December 9, 2025 04:51
Optional: true,
Description: "The region you want to attach the resource to",
Validators: []validator.String{
stringvalidator.OneOf(AllRegions()...),
Copy link
Member

Choose a reason for hiding this comment

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

I would like only a warning and not a blocker

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.

Add support for actions in cockpit

4 participants