-
Notifications
You must be signed in to change notification settings - Fork 8.3k
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
[UII] Convert deployment mode selector to radio group #208859
base: main
Are you sure you want to change the base?
Conversation
Pinging @elastic/fleet (Team:Fleet) |
💛 Build succeeded, but was flaky
Failed CI StepsTest Failures
Metrics [docs]Async chunks
History
cc @jen-huang |
defaultMessage="Agentless" | ||
/>{' '} | ||
<EuiBetaBadge | ||
label="Beta" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
a bit unrelated, I'm wondering when the Beta badge is planned to be removed?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me. Thanks for aligning the UIs
@@ -86,6 +86,11 @@ export function useSetupTechnology({ | |||
const orginalAgentPolicyRef = useRef<NewAgentPolicy>({ ...newAgentPolicy }); | |||
const [currentAgentPolicy, setCurrentAgentPolicy] = useState(newAgentPolicy); | |||
|
|||
const allowedSetupTechnologies = useMemo(() => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I like this change.
Summary
Closes https://github.com/elastic/ingest-dev/issues/4507. This PR converts the drop down selector in policy editor for
Agentless
/Agent-based
to a radio group.Closes #208861. This PR also fixes an issue where
Agent-based
option was still available even when integration only specifies agentless deployment mode.The selector used by the custom editor for CSPM integration was already converted to radios in #208645. This PR handles the conversion for the generic editor used by other agentless integrations such as
elastic_connectors
. It also makes minor formatting changes to the CSPM one to bring the two visually in line:Generic:
CSPM:
Checklist
Check the PR satisfies following conditions.
Reviewers should verify this PR satisfies this list as well.
release_note:*
label is applied per the guidelines