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

[Reporting/CSV Export] add the job ID in the x-opaque-id header #195332

Open
tsullivan opened this issue Oct 7, 2024 · 2 comments
Open

[Reporting/CSV Export] add the job ID in the x-opaque-id header #195332

tsullivan opened this issue Oct 7, 2024 · 2 comments
Labels
Feature:Reporting:CSV Reporting issues pertaining to CSV file export Team:SharedUX Team label for AppEx-SharedUX (formerly Global Experience)

Comments

@tsullivan
Copy link
Member

It would be nice to add the job ID in the x-opaque-id header or in any case being able to track all the logs associated to a given run.

When Kibana sends requests to Elasticsearch for CSV job execution, we should send the report job ID in the x-opaque-id header. This will make it easier to see what is happening on the ES side during report generation.

@tsullivan tsullivan added the Team:SharedUX Team label for AppEx-SharedUX (formerly Global Experience) label Oct 7, 2024
@elasticmachine
Copy link
Contributor

Pinging @elastic/appex-sharedux (Team:SharedUX)

@tsullivan tsullivan changed the title [Reporting/CSV Export] [Reporting/CSV Export] add the job ID in the x-opaque-id header Oct 10, 2024
@tsullivan
Copy link
Member Author

In addition to sending the job ID in the x-opaque-id header along with requests made to Elasticsearch, we should also update the way we add the job ID to the logging tags, to make it easier to search logs by job ID using the default analyzer, when logs are stored in Elasticsearch.

BEFORE const logger = this.logger.get(``execute-job:${jobId}``);
AFTER const logger = this.logger.get('execute-job', jobId);

@tsullivan tsullivan added the Feature:Reporting:CSV Reporting issues pertaining to CSV file export label Nov 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature:Reporting:CSV Reporting issues pertaining to CSV file export Team:SharedUX Team label for AppEx-SharedUX (formerly Global Experience)
Projects
None yet
Development

No branches or pull requests

2 participants