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

[OAS] Add is_system_action to connector responses #163969

Merged
merged 4 commits into from
Aug 17, 2023

Conversation

lcawl
Copy link
Contributor

@lcawl lcawl commented Aug 15, 2023

Summary

Relates to #160367

This PR updates the openAPI specification for connectors to include the new is_system_action property.

@github-actions
Copy link

Documentation preview:

@apmmachine
Copy link
Contributor

🤖 GitHub comments

Expand to view the GitHub comments

Just comment with:

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

@lcawl lcawl added Team:ResponseOps Label for the ResponseOps team (formerly the Cases and Alerting teams) v8.10.0 Feature:Actions/ConnectorsManagement Issues related to Connectors Management UX backport:skip This commit does not require backporting docs release_note:skip Skip the PR/issue when compiling release notes and removed docs labels Aug 15, 2023
@lcawl lcawl marked this pull request as ready for review August 15, 2023 21:32
@lcawl lcawl requested a review from a team as a code owner August 15, 2023 21:32
@lcawl lcawl requested a review from cnasikas August 15, 2023 21:32
@elasticmachine
Copy link
Contributor

Pinging @elastic/response-ops (Team:ResponseOps)

@lcawl lcawl requested review from adcoelho and removed request for cnasikas August 16, 2023 15:20
@kibana-ci
Copy link
Collaborator

💛 Build succeeded, but was flaky

Failed CI Steps

Metrics [docs]

✅ unchanged

History

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

Copy link
Contributor

@adcoelho adcoelho left a comment

Choose a reason for hiding this comment

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

What about components/schemas/connector_response_properties.yaml?

It is used in s@{spaceid}@api@actions@connector.yaml and it seems it might be missing the is_system_action property too. (The example response for that API also has it create_index_connector_response.yaml.)

@lcawl
Copy link
Contributor Author

lcawl commented Aug 17, 2023

What about components/schemas/connector_response_properties.yaml?

That file just points to all the connector-specific files, which contain the change, so I think we're good.

(The example response for that API also has it create_index_connector_response.yaml.)

Yes, that example has been updated and passes validation so it's correctly matching the spec.

@lcawl lcawl merged commit 5c1d118 into elastic:main Aug 17, 2023
24 checks passed
@lcawl lcawl deleted the system-action-oas branch August 17, 2023 20:14
@lcawl
Copy link
Contributor Author

lcawl commented Aug 23, 2023

💚 All backports created successfully

Status Branch Result
8.10

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

Questions ?

Please refer to the Backport tool documentation

lcawl added a commit to lcawl/kibana that referenced this pull request Aug 23, 2023
lcawl added a commit that referenced this pull request Aug 24, 2023
…164642)

# Backport

This will backport the following commits from `main` to `8.10`:
- [[OAS] Add is_system_action to connector responses
(#163969)](#163969)

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

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

<!--BACKPORT [{"author":{"name":"Lisa
Cawley","email":"lcawley@elastic.co"},"sourceCommit":{"committedDate":"2023-08-17T20:13:35Z","message":"[OAS]
Add is_system_action to connector responses
(#163969)","sha":"5c1d118c36a9d6cc572fb49326a9d059da73d1fc","branchLabelMapping":{"^v8.10.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","backport:skip","Team:ResponseOps","Feature:Actions/ConnectorsManagement","v8.10.0","v8.11.0"],"number":163969,"url":"#163969
Add is_system_action to connector responses
(#163969)","sha":"5c1d118c36a9d6cc572fb49326a9d059da73d1fc"}},"sourceBranch":"main","suggestedTargetBranches":[],"targetPullRequestStates":[{"branch":"main","label":"v8.10.0","labelRegex":"^v8.10.0$","isSourceBranch":true,"state":"MERGED","url":"#163969
Add is_system_action to connector responses
(#163969)","sha":"5c1d118c36a9d6cc572fb49326a9d059da73d1fc"}},{"branch":"8.11","label":"v8.11.0","labelRegex":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->
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 Feature:Actions/ConnectorsManagement Issues related to Connectors Management UX release_note:skip Skip the PR/issue when compiling release notes Team:ResponseOps Label for the ResponseOps team (formerly the Cases and Alerting teams) v8.10.0 v8.11.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants