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

[Cloud Security] Add AWS Org instructions #163952

Merged
merged 6 commits into from
Aug 16, 2023

Conversation

JordanSh
Copy link
Contributor

@JordanSh JordanSh commented Aug 15, 2023

Summary

Adding individual instructions based on account type

Before save instruction: AWS Org Account
image

Before save instruction: Single Account
image

Modal instructions (shared with flyout): AWS Org Account
image

Modal instructions (shared with flyout): Single Account
image

@JordanSh JordanSh added release_note:skip Skip the PR/issue when compiling release notes Team:Cloud Security Cloud Security team related v8.10.0 labels Aug 15, 2023
@JordanSh JordanSh self-assigned this Aug 15, 2023
@JordanSh JordanSh changed the title Add org instructions [Cloud Security] Add AWS Org instructions Aug 15, 2023
@JordanSh JordanSh marked this pull request as ready for review August 15, 2023 16:19
@JordanSh JordanSh requested review from a team as code owners August 15, 2023 16:19
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-cloud-security-posture (Team:Cloud Security)

@kibana-ci
Copy link
Collaborator

💚 Build Succeeded

Metrics [docs]

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
cloudSecurityPosture 259.6KB 260.0KB +432.0B
fleet 1022.7KB 1023.1KB +389.0B
total +821.0B

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

cc @JordanSh

Copy link
Contributor

@animehart animehart left a comment

Choose a reason for hiding this comment

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

lgtm

@botelastic botelastic bot added the Team:Fleet Team label for Observability Data Collection Fleet team label Aug 15, 2023
@elasticmachine
Copy link
Contributor

Pinging @elastic/fleet (Team:Fleet)

Copy link
Contributor

@opauloh opauloh left a comment

Choose a reason for hiding this comment

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

lgtm

<li>
<FormattedMessage
id="xpack.fleet.cloudFormation.guide.steps.organizationLogin"
defaultMessage="Log in as an admin in the management account of the AWS Organization you want to onboard"
Copy link
Contributor

Choose a reason for hiding this comment

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

any specific reason to have different wording here and in aws_credentials_form?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I'm not sure, one is before the "save and continue" button is pressed, the others are for the modals and flyouts.

@tinnytintin10 @einatjacoby any particular reason for not keeping the texts the same? or should we open a follow up pr?

Copy link
Contributor

Choose a reason for hiding this comment

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

I was just confused by the fact that for the Cloudformation option, two texts are identical Log in as an admin in the AWS account you want to onboard and Log in as an admin to the AWS Account you want to onboard (except the Account starting from a capital letter) and for the AWS Org texts differ but the meaning seems the same for me.
Not a big deal, I guess I just like consistency :)

Copy link
Contributor Author

@JordanSh JordanSh Aug 16, 2023

Choose a reason for hiding this comment

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

I'm 100% with you, I assume it was a mistake since those texts were changed last minute since the figma had some inaccuracies. I'll open a pr according to product decision.

@JordanSh JordanSh merged commit a90a14b into elastic:main Aug 16, 2023
@kibanamachine kibanamachine added the backport:skip This commit does not require backporting label Aug 16, 2023
hop-dev pushed a commit to hop-dev/kibana that referenced this pull request Aug 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport:skip This commit does not require backporting release_note:skip Skip the PR/issue when compiling release notes Team:Cloud Security Cloud Security team related Team:Fleet Team label for Observability Data Collection Fleet team v8.10.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants