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

docs: reorganize quickstart and home pages #1393

Open
wants to merge 16 commits into
base: develop
Choose a base branch
from

Conversation

annabog0
Copy link

@annabog0 annabog0 commented Oct 22, 2024

Issue number: https://splunk.atlassian.net/browse/ADDON-68816

Summary

Changes

Please provide a summary of what's being changed

User experience

Please describe what the user experience looks like before and after this change

Checklist

If your change doesn't seem to apply, please leave them unchecked.

@annabog0 annabog0 requested a review from a team as a code owner October 22, 2024 08:01
Copy link

github-actions bot commented Oct 22, 2024

All contributors have NOT signed the CLA Document


I have read the CLA Document and I hereby accept the CLA


2 out of 4 committers have signed the CLA.
✅ (kkedziak-splunk)[https://github.com/kkedziak-splunk]
✅ (artemrys)[https://github.com/artemrys]
@annabog0
@abogusz0
You can retrigger this bot by commenting recheck in this Pull Request. Posted by the CLA Assistant Lite bot.

@pull-request-size pull-request-size bot added size/L and removed size/M labels Oct 23, 2024
docs/generated_files.md Outdated Show resolved Hide resolved
docs/dot_conf_files.md Outdated Show resolved Hide resolved
docs/dot_conf_files.md Show resolved Hide resolved
docs/deprecated_content.md Outdated Show resolved Hide resolved
docs/caveats.md Outdated Show resolved Hide resolved
docs/build_add_on_from_existing_one.md Outdated Show resolved Hide resolved
docs/commands.md Outdated Show resolved Hide resolved
docs/commands.md Outdated Show resolved Hide resolved
docs/quickstart.md Show resolved Hide resolved
Comment on lines +17 to +18
If you use Windows, ...
If you use macOS, ...<!--- Artem to give more info--->
Copy link
Member

Choose a reason for hiding this comment

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

@kkedziak-splunk please provide the command on how to initialize the virtualenv.

@artemrys artemrys changed the title Update mkdocs.yml docs: reorganize quickstart and home pages Nov 5, 2024
docs/commands.md Outdated

Apart from standard files needed for the add-on, it also adds search head
clustering files in the `default/server.conf` file and reload triggers in the
`default/app.conf` file. Those files will be soon <!--- verb is missing ---> by generated automatically by the
Copy link

Choose a reason for hiding this comment

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

Those files will be soon <!--- verb is missing ---> by generated automatically by the ucc-gen build command itself. @artemrys what will happen the files?

Copy link
Member

Choose a reason for hiding this comment

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

I think "generated", it's just an extra "by" there.

Comment on lines 21 to 22
Also the `triggers` stanza is created by `ucc-gen`. It <!--- it stands for stanza or ucc-gen---> determines what
the `.conf` files are used in the add-on and generates the relevant key-value
Copy link

Choose a reason for hiding this comment

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

@artemrys In this sentence: "It determines what the .conf files are used..." does "It" stands for stanza or ucc-gen?

Copy link
Member

Choose a reason for hiding this comment

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

Yes, UCC (as a framework) or to be more specific ucc-gen build as a specific command.

* it possibly extends the UI with custom codes (for more information, see [here](custom_ui_extensions/custom_hook.md)).
* it possibly extends the build process via a `additional_packaging.py` file (more information, [here](additional_packaging.md)).
* generates the necessary files defined for the Alert Action, if defined in globalConfig (for more informaiton, see [here](alert_actions/index.md)).
## What UCC generates
Copy link

Choose a reason for hiding this comment

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

I added the caveats as a note in this section. @artemrys do we leave this info here?

Copy link
Member

Choose a reason for hiding this comment

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

Yes, let's leave it for now, that's not a deal breaker for me.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants