Skip to content

(Doc+) Alerts UI cannot be CCS#5513

Merged
joepeeples merged 3 commits into
elastic:mainfrom
stefnestor:patch-2
Jul 17, 2024
Merged

(Doc+) Alerts UI cannot be CCS#5513
joepeeples merged 3 commits into
elastic:mainfrom
stefnestor:patch-2

Conversation

@stefnestor

Copy link
Copy Markdown
Member

👋 howdy, team!

Playing forward a couple ballpark Githubs (kindly see their sub-internal-links) #4653 , elastic/kibana#172534 , can we expand this doc's note to more heavily indicate even CCS is not possible on the Alerts UI?

👋 howdy, team! 

Playing forward a couple ballpark Githubs (kindly see their sub-internal-links) elastic#4653 , elastic/kibana#172534 , can we expand [this doc's](https://www.elastic.co/guide/en/security/master/data-views-in-sec.html#_create_or_modify_a_data_view) note to more heavily indicate even CCS is not possible on the Alerts UI?
@stefnestor
stefnestor requested a review from a team as a code owner July 5, 2024 17:31
@github-actions

github-actions Bot commented Jul 5, 2024

Copy link
Copy Markdown

A documentation preview will be available soon.

Request a new doc build by commenting
  • Rebuild this PR: run docs-build
  • Rebuild this PR and all Elastic docs: run docs-build rebuild

run docs-build is much faster than run docs-build rebuild. A rebuild should only be needed in rare situations.

If your PR continues to fail for an unknown reason, the doc build pipeline may be broken. Elastic employees can check the pipeline status here.

@mergify

mergify Bot commented Jul 5, 2024

Copy link
Copy Markdown
Contributor

This pull request does not have a backport label. Could you fix it @stefnestor? 🙏
To fixup this pull request, you need to add the backport labels for the needed
branches, such as:

  • v7.x is the label to automatically backport to the 7.x branch.
  • v7./d./d is the label to automatically backport to the 7./d branch. /d is the digit

NOTE: backport-skip has been added to this pull request.

@mergify mergify Bot added the backport-skip label Jul 5, 2024

@joepeeples joepeeples 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.

Thanks for the PR @stefnestor! I added a suggested edit, LMK what you think or feel free to commit if it looks OK to you.

We should also update the serverless docs with the same clarification; I'll open a separate PR to do that.

Comment thread docs/getting-started/data-views-in-sec.asciidoc Outdated
joepeeples and others added 2 commits July 17, 2024 15:32
Co-authored-by: Joe Peeples <joe.peeples@elastic.co>
@joepeeples

Copy link
Copy Markdown
Contributor

We should also update the serverless docs with the same clarification; I can push a commit to this PR that'll do that shortly.

@stefnestor As I dug into this more, I noticed some other quirks (mostly unrelated) that I also need to fix, so I'll open a separate PR to update serverless docs and do the other fixes in one swoop. In the meantime, I'll approve this PR and can take care of merging & backporting to keep things moving.

@joepeeples joepeeples 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.

👍

@mergify mergify Bot removed the backport-skip label Jul 17, 2024
@joepeeples joepeeples added v8.14.0 enhancement New feature or request Feature: Alerts Docset: ESS Issues that apply to docs in the Stack release labels Jul 17, 2024
@joepeeples
joepeeples merged commit 5c199b0 into elastic:main Jul 17, 2024
mergify Bot pushed a commit that referenced this pull request Jul 17, 2024
* (Doc+) Alerts UI cannot be CCS

👋 howdy, team!

Playing forward a couple ballpark Githubs (kindly see their sub-internal-links) #4653 , elastic/kibana#172534 , can we expand [this doc's](https://www.elastic.co/guide/en/security/master/data-views-in-sec.html#_create_or_modify_a_data_view) note to more heavily indicate even CCS is not possible on the Alerts UI?

* feedback

Co-authored-by: Joe Peeples <joe.peeples@elastic.co>

---------

Co-authored-by: Joe Peeples <joe.peeples@elastic.co>
(cherry picked from commit 5c199b0)
mergify Bot pushed a commit that referenced this pull request Jul 17, 2024
* (Doc+) Alerts UI cannot be CCS

👋 howdy, team!

Playing forward a couple ballpark Githubs (kindly see their sub-internal-links) #4653 , elastic/kibana#172534 , can we expand [this doc's](https://www.elastic.co/guide/en/security/master/data-views-in-sec.html#_create_or_modify_a_data_view) note to more heavily indicate even CCS is not possible on the Alerts UI?

* feedback

Co-authored-by: Joe Peeples <joe.peeples@elastic.co>

---------

Co-authored-by: Joe Peeples <joe.peeples@elastic.co>
(cherry picked from commit 5c199b0)
joepeeples added a commit that referenced this pull request Jul 17, 2024
Serverless follow up to #5513
joepeeples pushed a commit that referenced this pull request Jul 17, 2024
* (Doc+) Alerts UI cannot be CCS

👋 howdy, team!

Playing forward a couple ballpark Githubs (kindly see their sub-internal-links) #4653 , elastic/kibana#172534 , can we expand [this doc's](https://www.elastic.co/guide/en/security/master/data-views-in-sec.html#_create_or_modify_a_data_view) note to more heavily indicate even CCS is not possible on the Alerts UI?

* feedback

Co-authored-by: Joe Peeples <joe.peeples@elastic.co>

---------

Co-authored-by: Joe Peeples <joe.peeples@elastic.co>
(cherry picked from commit 5c199b0)

Co-authored-by: Stef Nestor <26751266+stefnestor@users.noreply.github.com>
joepeeples pushed a commit that referenced this pull request Jul 17, 2024
* (Doc+) Alerts UI cannot be CCS

👋 howdy, team!

Playing forward a couple ballpark Githubs (kindly see their sub-internal-links) #4653 , elastic/kibana#172534 , can we expand [this doc's](https://www.elastic.co/guide/en/security/master/data-views-in-sec.html#_create_or_modify_a_data_view) note to more heavily indicate even CCS is not possible on the Alerts UI?

* feedback

Co-authored-by: Joe Peeples <joe.peeples@elastic.co>

---------

Co-authored-by: Joe Peeples <joe.peeples@elastic.co>
(cherry picked from commit 5c199b0)

Co-authored-by: Stef Nestor <26751266+stefnestor@users.noreply.github.com>
joepeeples added a commit that referenced this pull request Jul 19, 2024
* Add admonition to serverless

Serverless follow up to #5513

* Update serverless to match ESS

Serverless follow up to #4695

* Fix URL variable/attribute

Should use `kibana-ref` instead of apm-app-ref since this has nothing to do with APM

* No-op change to trigger build

(i hope)
mergify Bot pushed a commit that referenced this pull request Jul 19, 2024
* Add admonition to serverless

Serverless follow up to #5513

* Update serverless to match ESS

Serverless follow up to #4695

* Fix URL variable/attribute

Should use `kibana-ref` instead of apm-app-ref since this has nothing to do with APM

* No-op change to trigger build

(i hope)

(cherry picked from commit 9e7d734)

# Conflicts:
#	docs/serverless/explore/data-views-in-sec.mdx
mergify Bot pushed a commit that referenced this pull request Jul 19, 2024
* Add admonition to serverless

Serverless follow up to #5513

* Update serverless to match ESS

Serverless follow up to #4695

* Fix URL variable/attribute

Should use `kibana-ref` instead of apm-app-ref since this has nothing to do with APM

* No-op change to trigger build

(i hope)

(cherry picked from commit 9e7d734)

# Conflicts:
#	docs/serverless/explore/data-views-in-sec.mdx
joepeeples added a commit that referenced this pull request Jul 19, 2024
…kport #5582) (#5586)

* (Doc+) Alerts UI cannot be CCS [serverless] + bonus fixes (#5582)

* Add admonition to serverless

Serverless follow up to #5513

* Update serverless to match ESS

Serverless follow up to #4695

* Fix URL variable/attribute

Should use `kibana-ref` instead of apm-app-ref since this has nothing to do with APM

* No-op change to trigger build

(i hope)

(cherry picked from commit 9e7d734)

# Conflicts:
#	docs/serverless/explore/data-views-in-sec.mdx

* Delete docs/serverless directory and its contents

---------

Co-authored-by: Joe Peeples <joe.peeples@elastic.co>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
joepeeples added a commit that referenced this pull request Jul 19, 2024
…kport #5582) (#5585)

* (Doc+) Alerts UI cannot be CCS [serverless] + bonus fixes (#5582)

* Add admonition to serverless

Serverless follow up to #5513

* Update serverless to match ESS

Serverless follow up to #4695

* Fix URL variable/attribute

Should use `kibana-ref` instead of apm-app-ref since this has nothing to do with APM

* No-op change to trigger build

(i hope)

(cherry picked from commit 9e7d734)

# Conflicts:
#	docs/serverless/explore/data-views-in-sec.mdx

* Delete docs/serverless directory and its contents

---------

Co-authored-by: Joe Peeples <joe.peeples@elastic.co>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Docset: ESS Issues that apply to docs in the Stack release enhancement New feature or request Feature: Alerts Feature: Data Views v8.14.0 v8.15.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants