Skip to content

Add Elastic-Api-Version header to kibana client#148

Merged
pchila merged 2 commits intoelastic:mainfrom
pchila:add-kibana-api-version
Sep 11, 2023
Merged

Add Elastic-Api-Version header to kibana client#148
pchila merged 2 commits intoelastic:mainfrom
pchila:add-kibana-api-version

Conversation

@pchila
Copy link
Copy Markdown
Member

@pchila pchila commented Sep 8, 2023

What does this PR do?

Adds the Elastic-Api-Version header to all request to kibana Api as it's mandatory since merge of elastic/kibana#163570

This is the first step of a 2-step fox for issue elastic/elastic-agent#3362: after this PR is merged we need to bump the version and use the newer version in elastic-agent.

Why is it important?

Checklist

  • [ ] My code follows the style guidelines of this project
  • [ ] I have commented my code, particularly in hard-to-understand areas
  • [ ] I have added tests that prove my fix is effective or that my feature works
  • [ ] I have added an entry in CHANGELOG.md

Author's Checklist

  • [ ]

Related issues

@pchila pchila added bug Something isn't working enhancement New feature or request labels Sep 8, 2023
@pchila pchila requested a review from a team as a code owner September 8, 2023 14:24
@pchila pchila self-assigned this Sep 8, 2023
@pchila pchila requested review from belimawr and leehinman and removed request for a team September 8, 2023 14:25
@pchila pchila added the Team:Elastic-Agent Label for the Agent team label Sep 8, 2023
@pchila pchila force-pushed the add-kibana-api-version branch from c098a12 to be1611e Compare September 8, 2023 14:27
@pchila pchila requested a review from leehinman September 8, 2023 15:15
@elasticmachine
Copy link
Copy Markdown
Collaborator

💚 Build Succeeded

History

cc @pchila

Copy link
Copy Markdown
Contributor

@leehinman leehinman left a comment

Choose a reason for hiding this comment

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

LGTM

@pchila pchila merged commit 9b36953 into elastic:main Sep 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working enhancement New feature or request Team:Elastic-Agent Label for the Agent team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants