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

[UII] Fix client-side validation for agent policy timeout fields #191674

Merged
merged 1 commit into from
Aug 28, 2024

Conversation

jen-huang
Copy link
Contributor

@jen-huang jen-huang commented Aug 28, 2024

Summary

Resolves #191583. Adjusts the client-side validation for agent policy timeout fields so that it correctly returns an error when non-numeric input is encountered, or when input is 0.

@jen-huang jen-huang added release_note:skip Skip the PR/issue when compiling release notes Team:Fleet Team label for Observability Data Collection Fleet team backport:prev-minor Backport to (8.x) the previous minor version (i.e. one version back from main) labels Aug 28, 2024
@jen-huang jen-huang self-assigned this Aug 28, 2024
@jen-huang jen-huang requested a review from a team as a code owner August 28, 2024 19:10
@elasticmachine
Copy link
Contributor

Pinging @elastic/fleet (Team:Fleet)

@obltmachine
Copy link

🤖 GitHub comments

Expand to view the GitHub comments

Just comment with:

  • /oblt-deploy : Deploy a Kibana instance using the Observability test environments.
  • run docs-build : Re-trigger the docs validation. (use unformatted text in the comment!)

@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
fleet 1.8MB 1.8MB +42.0B

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

cc @jen-huang

Copy link
Member

@nchaulet nchaulet left a comment

Choose a reason for hiding this comment

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

code LGTM 🚀

@jen-huang jen-huang merged commit f29bf1c into elastic:main Aug 28, 2024
26 checks passed
@jen-huang jen-huang deleted the fix/191583 branch August 28, 2024 21:05
@kibanamachine
Copy link
Contributor

💔 All backports failed

Status Branch Result
8.15 Backport failed because of merge conflicts

You might need to backport the following PRs to 8.15:
- [Detection Engine] Update toast message for adding exception (#191032)

Manual backport

To create the backport manually run:

node scripts/backport --pr 191674

Questions ?

Please refer to the Backport tool documentation

@jen-huang
Copy link
Contributor Author

💚 All backports created successfully

Status Branch Result
8.15

Note: Successful backport PRs will be merged automatically after passing CI.

Questions ?

Please refer to the Backport tool documentation

jen-huang added a commit to jen-huang/kibana that referenced this pull request Aug 29, 2024
…stic#191674)

## Summary

Resolves elastic#191583. Adjusts the client-side validation for agent policy
timeout fields so that it correctly returns an error when non-numeric
input is encountered, or when input is 0.

(cherry picked from commit f29bf1c)

# Conflicts:
#	x-pack/plugins/translations/translations/fr-FR.json
#	x-pack/plugins/translations/translations/ja-JP.json
#	x-pack/plugins/translations/translations/zh-CN.json
jen-huang added a commit that referenced this pull request Aug 29, 2024
…ds (#191674) (#191752)

# Backport

This will backport the following commits from `main` to `8.15`:
- [[UII] Fix client-side validation for agent policy timeout fields
(#191674)](#191674)

<!--- Backport version: 8.9.8 -->

### Questions ?
Please refer to the [Backport tool
documentation](https://github.com/sqren/backport)

<!--BACKPORT [{"author":{"name":"Jen
Huang","email":"[email protected]"},"sourceCommit":{"committedDate":"2024-08-28T21:05:11Z","message":"[UII]
Fix client-side validation for agent policy timeout fields
(#191674)\n\n## Summary\r\n\r\nResolves #191583. Adjusts the client-side
validation for agent policy\r\ntimeout fields so that it correctly
returns an error when non-numeric\r\ninput is encountered, or when input
is
0.","sha":"f29bf1c3113de90fc0ca08bf7a59a33d53638940","branchLabelMapping":{"^v8.16.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","Team:Fleet","backport:prev-minor","v8.16.0"],"number":191674,"url":"https://github.com/elastic/kibana/pull/191674","mergeCommit":{"message":"[UII]
Fix client-side validation for agent policy timeout fields
(#191674)\n\n## Summary\r\n\r\nResolves #191583. Adjusts the client-side
validation for agent policy\r\ntimeout fields so that it correctly
returns an error when non-numeric\r\ninput is encountered, or when input
is
0.","sha":"f29bf1c3113de90fc0ca08bf7a59a33d53638940"}},"sourceBranch":"main","suggestedTargetBranches":[],"targetPullRequestStates":[{"branch":"main","label":"v8.16.0","labelRegex":"^v8.16.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/191674","number":191674,"mergeCommit":{"message":"[UII]
Fix client-side validation for agent policy timeout fields
(#191674)\n\n## Summary\r\n\r\nResolves #191583. Adjusts the client-side
validation for agent policy\r\ntimeout fields so that it correctly
returns an error when non-numeric\r\ninput is encountered, or when input
is 0.","sha":"f29bf1c3113de90fc0ca08bf7a59a33d53638940"}}]}] BACKPORT-->
jen-huang added a commit to jen-huang/kibana that referenced this pull request Sep 27, 2024
jen-huang added a commit that referenced this pull request Sep 30, 2024
…lds (#191674)" (#194338)

## Summary

Resolves #194221.

This PR reverts commit f29bf1c. Timeout
fields should be clearable with undefined value or zero.
kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Sep 30, 2024
…lds (elastic#191674)" (elastic#194338)

## Summary

Resolves elastic#194221.

This PR reverts commit f29bf1c. Timeout
fields should be clearable with undefined value or zero.

(cherry picked from commit 6792cb4)
kibanamachine added a commit that referenced this pull request Sep 30, 2024
…timeout fields (#191674)&quot; (#194338) (#194491)

# Backport

This will backport the following commits from `main` to `8.x`:
- [Revert &quot;[UII] Fix client-side validation for agent policy
timeout fields (#191674)&quot;
(#194338)](#194338)

<!--- Backport version: 9.4.3 -->

### Questions ?
Please refer to the [Backport tool
documentation](https://github.com/sqren/backport)

<!--BACKPORT [{"author":{"name":"Jen
Huang","email":"[email protected]"},"sourceCommit":{"committedDate":"2024-09-30T21:22:37Z","message":"Revert
\"[UII] Fix client-side validation for agent policy timeout fields
(#191674)\" (#194338)\n\n## Summary\r\n\r\nResolves #194221.\r\n\r\nThis
PR reverts commit f29bf1c.
Timeout\r\nfields should be clearable with undefined value or
zero.","sha":"6792cb4dca5adecf1a48a4d052dae603db5772e3","branchLabelMapping":{"^v9.0.0$":"main","^v8.16.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:fix","Team:Fleet","v9.0.0","backport:prev-major"],"title":"Revert
\"[UII] Fix client-side validation for agent policy timeout fields
(#191674)\"","number":194338,"url":"https://github.com/elastic/kibana/pull/194338","mergeCommit":{"message":"Revert
\"[UII] Fix client-side validation for agent policy timeout fields
(#191674)\" (#194338)\n\n## Summary\r\n\r\nResolves #194221.\r\n\r\nThis
PR reverts commit f29bf1c.
Timeout\r\nfields should be clearable with undefined value or
zero.","sha":"6792cb4dca5adecf1a48a4d052dae603db5772e3"}},"sourceBranch":"main","suggestedTargetBranches":[],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","branchLabelMappingKey":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/194338","number":194338,"mergeCommit":{"message":"Revert
\"[UII] Fix client-side validation for agent policy timeout fields
(#191674)\" (#194338)\n\n## Summary\r\n\r\nResolves #194221.\r\n\r\nThis
PR reverts commit f29bf1c.
Timeout\r\nfields should be clearable with undefined value or
zero.","sha":"6792cb4dca5adecf1a48a4d052dae603db5772e3"}}]}] BACKPORT-->

Co-authored-by: Jen Huang <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport:prev-minor Backport to (8.x) the previous minor version (i.e. one version back from main) release_note:skip Skip the PR/issue when compiling release notes Team:Fleet Team label for Observability Data Collection Fleet team v8.15.1 v8.16.0
Projects
None yet
6 participants