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

[Automatic Import] Fix yarn draw-graphs command #198229

Merged
merged 6 commits into from
Nov 1, 2024

Conversation

ilyannn
Copy link
Contributor

@ilyannn ilyannn commented Oct 29, 2024

Summary

Fixes #196425.

It turns out the reason yarn draw-graphs produced the three-box graphs was because .withConfig creates an instance of RunnableInput which does not have a good way to draw itself other than as three boxes.

The solution was to makes sure we are calling the original version without .withConfig when drawing the graphs. We still call the new version when invoking them, as demonstrated by the run names here:

image

We are now able to generate the correct graphs for all chains.

Test result: ai_postgres_202410300002-1.0.0.zip

Checklist

@ilyannn ilyannn added release_note:skip Skip the PR/issue when compiling release notes backport:prev-minor Backport to (8.x) the previous minor version (i.e. one version back from main) Team:Security-Scalability Team label for Security Integrations Scalability Team Feature:AutomaticImport labels Oct 29, 2024
@ilyannn ilyannn self-assigned this Oct 29, 2024
@ilyannn ilyannn marked this pull request as ready for review October 29, 2024 23:26
@ilyannn ilyannn requested a review from a team as a code owner October 29, 2024 23:26
@elasticmachine
Copy link
Contributor

Pinging @elastic/security-scalability (Team:Security-Scalability)

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

✅ unchanged

cc @ilyannn

Copy link
Contributor

@kgeller kgeller left a comment

Choose a reason for hiding this comment

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

LGTM

@ilyannn
Copy link
Contributor Author

ilyannn commented Nov 1, 2024

@elasticmachine merge upstream

@ilyannn ilyannn merged commit 0ecef0a into elastic:main Nov 1, 2024
22 checks passed
@ilyannn ilyannn deleted the auto-import/fix-draw-graphs branch November 1, 2024 13:38
@kibanamachine
Copy link
Contributor

Starting backport for target branches: 8.x

https://github.com/elastic/kibana/actions/runs/11630261121

kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Nov 1, 2024
## Summary

Fixes elastic#196425.

It turns out the reason `yarn draw-graphs` produced the three-box graphs
was because `.withConfig` creates an instance of RunnableInput which does
not have a good way to draw itself other than as three boxes.

The solution was to makes sure we are calling the original version
without `.withConfig` when drawing the graphs. We still call the new
version when invoking them, as demonstrated by the run names here.

We are now able to generate the correct graphs for all chains.

---------

Co-authored-by: Elastic Machine <[email protected]>
(cherry picked from commit 0ecef0a)
@kibanamachine
Copy link
Contributor

💚 All backports created successfully

Status Branch Result
8.x

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

Questions ?

Please refer to the Backport tool documentation

kibanamachine added a commit that referenced this pull request Nov 1, 2024
)

# Backport

This will backport the following commits from `main` to `8.x`:
- [[Automatic Import] Fix yarn draw-graphs command
(#198229)](#198229)

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

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

<!--BACKPORT [{"author":{"name":"Ilya
Nikokoshev","email":"[email protected]"},"sourceCommit":{"committedDate":"2024-11-01T13:38:07Z","message":"[Automatic
Import] Fix yarn draw-graphs command (#198229)\n\n##
Summary\r\n\r\nFixes
https://github.com/elastic/kibana/issues/196425.\r\n\r\nIt turns out the
reason `yarn draw-graphs` produced the three-box graphs\r\nwas because
`.withConfig` creates an instance of RunnableInput which does \r\nnot
have a good way to draw itself other than as three boxes.\r\n\r\nThe
solution was to makes sure we are calling the original
version\r\nwithout `.withConfig` when drawing the graphs. We still call
the new\r\nversion when invoking them, as demonstrated by the run names
here.\r\n\r\nWe are now able to generate the correct graphs for all
chains.\r\n\r\n---------\r\n\r\nCo-authored-by: Elastic Machine
<[email protected]>","sha":"0ecef0ab60db28d41da69a414f7db31f278261ab","branchLabelMapping":{"^v9.0.0$":"main","^v8.17.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","v9.0.0","backport:prev-minor","Team:Security-Scalability","Feature:AutomaticImport"],"title":"[Automatic
Import] Fix yarn draw-graphs
command","number":198229,"url":"https://github.com/elastic/kibana/pull/198229","mergeCommit":{"message":"[Automatic
Import] Fix yarn draw-graphs command (#198229)\n\n##
Summary\r\n\r\nFixes
https://github.com/elastic/kibana/issues/196425.\r\n\r\nIt turns out the
reason `yarn draw-graphs` produced the three-box graphs\r\nwas because
`.withConfig` creates an instance of RunnableInput which does \r\nnot
have a good way to draw itself other than as three boxes.\r\n\r\nThe
solution was to makes sure we are calling the original
version\r\nwithout `.withConfig` when drawing the graphs. We still call
the new\r\nversion when invoking them, as demonstrated by the run names
here.\r\n\r\nWe are now able to generate the correct graphs for all
chains.\r\n\r\n---------\r\n\r\nCo-authored-by: Elastic Machine
<[email protected]>","sha":"0ecef0ab60db28d41da69a414f7db31f278261ab"}},"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/198229","number":198229,"mergeCommit":{"message":"[Automatic
Import] Fix yarn draw-graphs command (#198229)\n\n##
Summary\r\n\r\nFixes
https://github.com/elastic/kibana/issues/196425.\r\n\r\nIt turns out the
reason `yarn draw-graphs` produced the three-box graphs\r\nwas because
`.withConfig` creates an instance of RunnableInput which does \r\nnot
have a good way to draw itself other than as three boxes.\r\n\r\nThe
solution was to makes sure we are calling the original
version\r\nwithout `.withConfig` when drawing the graphs. We still call
the new\r\nversion when invoking them, as demonstrated by the run names
here.\r\n\r\nWe are now able to generate the correct graphs for all
chains.\r\n\r\n---------\r\n\r\nCo-authored-by: Elastic Machine
<[email protected]>","sha":"0ecef0ab60db28d41da69a414f7db31f278261ab"}}]}]
BACKPORT-->

Co-authored-by: Ilya Nikokoshev <[email protected]>
nreese pushed a commit to nreese/kibana that referenced this pull request Nov 1, 2024
## Summary

Fixes elastic#196425.

It turns out the reason `yarn draw-graphs` produced the three-box graphs
was because `.withConfig` creates an instance of RunnableInput which does 
not have a good way to draw itself other than as three boxes.

The solution was to makes sure we are calling the original version
without `.withConfig` when drawing the graphs. We still call the new
version when invoking them, as demonstrated by the run names here.

We are now able to generate the correct graphs for all chains.

---------

Co-authored-by: Elastic Machine <[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) Feature:AutomaticImport release_note:skip Skip the PR/issue when compiling release notes Team:Security-Scalability Team label for Security Integrations Scalability Team v8.17.0 v9.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Automatic Import] Bug: broken yarn draw-graphs
4 participants