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

[EDR Workflows][Osquery] Missing timeout value when running query from query history #174088

Merged
merged 3 commits into from
Jan 22, 2024

Conversation

szwarckonrad
Copy link
Contributor

@szwarckonrad szwarckonrad commented Jan 2, 2024

#174082

Fixed bug with timeout field not being properly passed from Query History list to query form.
Added test coverage.

Screen.Recording.2024-01-02.at.15.24.26.mov

@szwarckonrad szwarckonrad added bug Fixes for quality problems that affect the customer experience release_note:skip Skip the PR/issue when compiling release notes Team:Defend Workflows “EDR Workflows” sub-team of Security Solution Osquery v8.12.1 labels Jan 2, 2024
@szwarckonrad szwarckonrad self-assigned this Jan 2, 2024
@szwarckonrad
Copy link
Contributor Author

/ci

@szwarckonrad szwarckonrad marked this pull request as ready for review January 22, 2024 09:21
@szwarckonrad szwarckonrad requested a review from a team as a code owner January 22, 2024 09:21
@elasticmachine
Copy link
Contributor

Pinging @elastic/security-defend-workflows (Team:Defend Workflows)

@szwarckonrad szwarckonrad changed the title [EDR Workflows][Osquery][AWAITS 8.12] Missing timeout value when running query from query history [EDR Workflows][Osquery] Missing timeout value when running query from query history Jan 22, 2024
# Conflicts:
#	x-pack/plugins/osquery/cypress/tasks/saved_queries.ts
@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
osquery 1.0MB 1.0MB +143.0B

History

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

cc @szwarckonrad

@szwarckonrad szwarckonrad merged commit 0aadaa9 into elastic:main Jan 22, 2024
47 checks passed
@kibanamachine
Copy link
Contributor

💔 All backports failed

Status Branch Result
8.12 Backport failed because of merge conflicts

Manual backport

To create the backport manually run:

node scripts/backport --pr 174088

Questions ?

Please refer to the Backport tool documentation

@szwarckonrad
Copy link
Contributor Author

💚 All backports created successfully

Status Branch Result
8.12

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

Questions ?

Please refer to the Backport tool documentation

szwarckonrad added a commit that referenced this pull request Jan 22, 2024
…ery from query history (#174088) (#175217)

# Backport

This will backport the following commits from `main` to `8.12`:
- [[EDR Workflows][Osquery] Missing timeout value when running query
from query history
(#174088)](#174088)

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

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

<!--BACKPORT [{"author":{"name":"Konrad
Szwarc","email":"[email protected]"},"sourceCommit":{"committedDate":"2024-01-22T14:17:12Z","message":"[EDR
Workflows][Osquery] Missing timeout value when running query from query
history
(#174088)\n\nhttps://github.com//issues/174082\r\n\r\nFixed
bug with `timeout` field not being properly passed from Query\r\nHistory
list to query form.\r\nAdded test coverage.\r\n
\r\n\r\nhttps://github.com/elastic/kibana/assets/29123534/48862490-d308-47c6-bdc1-d2c10ec8533d","sha":"0aadaa9d7cdc8c1b1f2c49731169b767176b9377","branchLabelMapping":{"^v8.13.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["bug","release_note:skip","Team:Defend
Workflows","ci:all-cypress-suites","Osquery","v8.12.1","v8.13.0"],"number":174088,"url":"https://github.com/elastic/kibana/pull/174088","mergeCommit":{"message":"[EDR
Workflows][Osquery] Missing timeout value when running query from query
history
(#174088)\n\nhttps://github.com//issues/174082\r\n\r\nFixed
bug with `timeout` field not being properly passed from Query\r\nHistory
list to query form.\r\nAdded test coverage.\r\n
\r\n\r\nhttps://github.com/elastic/kibana/assets/29123534/48862490-d308-47c6-bdc1-d2c10ec8533d","sha":"0aadaa9d7cdc8c1b1f2c49731169b767176b9377"}},"sourceBranch":"main","suggestedTargetBranches":["8.12"],"targetPullRequestStates":[{"branch":"8.12","label":"v8.12.1","labelRegex":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"main","label":"v8.13.0","labelRegex":"^v8.13.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/174088","number":174088,"mergeCommit":{"message":"[EDR
Workflows][Osquery] Missing timeout value when running query from query
history
(#174088)\n\nhttps://github.com//issues/174082\r\n\r\nFixed
bug with `timeout` field not being properly passed from Query\r\nHistory
list to query form.\r\nAdded test coverage.\r\n
\r\n\r\nhttps://github.com/elastic/kibana/assets/29123534/48862490-d308-47c6-bdc1-d2c10ec8533d","sha":"0aadaa9d7cdc8c1b1f2c49731169b767176b9377"}}]}]
BACKPORT-->
CoenWarmer pushed a commit to CoenWarmer/kibana that referenced this pull request Feb 15, 2024
…m query history (elastic#174088)

elastic#174082

Fixed bug with `timeout` field not being properly passed from Query
History list to query form.
Added test coverage.
 

https://github.com/elastic/kibana/assets/29123534/48862490-d308-47c6-bdc1-d2c10ec8533d
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Fixes for quality problems that affect the customer experience ci:all-cypress-suites Osquery release_note:skip Skip the PR/issue when compiling release notes Team:Defend Workflows “EDR Workflows” sub-team of Security Solution v8.12.1 v8.13.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants