feat: introduce package command #761
Closed
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* refactor: generate data/ui/* files instead of including them automatically This commit has failing smoke tests, but the changes are minor. * feat: update UCC UI to v1.25.0 * test: fixing test failures * test: fixing smoke tests * chore: remove dead code * chore: downgrade charset-normalizer to 3.0.1 to make FOSSA happy
This PR gives `global_config.GlobalConfig` class ability to check if OAuth is configured in the `global_config` file.
* refactor: move get_os_path out to utils * refactor: get rid of PARENT_DIR in build * refactor: move recursive_overwrite to utils * refactor: move app_manifest read logic to a separate function
* refactor: alert code * test: fix smoke test * refactor: clean up some code * refactor: simplify alert generation code * refactor: alert actions HTML generation * refactor: remove some dead code in alert actions .conf generation * refactor: remove dead code from alert actions Python files generation * refactor: move all alert building to a separate module * refactor: simplified normalize function for alert actions * test: add tests for modular_alert_builder.builder * chore: raise coverage bar to 65
From now on, we are not going to support comments in app.manifest file.
* feat: app.manifest is being validated now according to the docs Docs link: https://dev.splunk.com/enterprise/reference/packagingtoolkit/pkgtoolkitappmanifest/#JSON-schema-200 `schemaVersion`, `targetWorkloads` and `supportedDeployments` keys are now being validated. * test: fix smoke tests
* fix: don't sort yaml as part of the dump sometimes developers prefer a certain, non alphabetic structure to their definition. generally, we should not modify the globalConfig during the build in my opinion. It is unexpected by the developer * test: adjust test_dump_yaml_config --------- Co-authored-by: Artem Rys <rysartem@gmail.com>
* refacfor: import_declare_test is not a variable anymore * refactor: get rid of global_config_post_processor * refactor: simplify global_config_builder_schema.py * test: cover get_j2_env function
* feat: initial implementation of dashboard * test: rename one of the test folders with add-on * test: adjust smoke tests * feat: enable dashboard in init by default * docs: initial documentation for dashboard * feat: use new functions from solnlib * feat: add a panel for events ingested by sourcetype * feat: add dashboard row with errors in the add-on * test: increase fail_under to 75 * test: rename one test * chore: rollback fail_under to 70
|
See #846 |
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.