Skip to content

feat: Sigma rules pt.2#467

Draft
MichaelThamm wants to merge 3 commits into
feat/sigma-rulesfrom
feat/sigma-rules-2
Draft

feat: Sigma rules pt.2#467
MichaelThamm wants to merge 3 commits into
feat/sigma-rulesfrom
feat/sigma-rules-2

Conversation

@MichaelThamm

@MichaelThamm MichaelThamm commented May 8, 2026

Copy link
Copy Markdown
Contributor

Issue

Add the testing for the prior PR:

Builds on this PR:

Solution

Add the feature.

Checklist

  • I have added or updated relevant documentation.
  • PR title makes an appropriate release note and follows conventional commits syntax.
  • Merge target is the correct branch, and relevant tandem backport PRs opened.
  • This change warrants a release, so I have updated the project.version field in the pyproject.toml file.
  • Some questions to answer:
    • Are we even validating rules in OTLP today? .add doesn't have it in its call path

Context

Testing Instructions

Testing of the SigmaRules class is achieved in:

This PR adds tests for the charmlib-scoped responsibilities.

Upgrade Notes

Co-authored-by: Copilot <copilot@github.com>
@MichaelThamm MichaelThamm mentioned this pull request May 8, 2026
4 tasks
labels = rule.get('labels', {})
assert labels.get('juju_model') == MODEL_NAME
assert labels.get('juju_model_uuid') == MODEL_UUID
assert labels.get('juju_application') == 'otlp-provider'

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Should this be otlp-requirer?

build-backend = "hatchling.build"

[tool.hatch.metadata]
allow-direct-references = true

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Remove when done testing

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.

1 participant