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

Push the total request-body-bytes to usage-api #194429

Draft
wants to merge 17 commits into
base: main
Choose a base branch
from

Conversation

ersin-erdal
Copy link
Contributor

Resolves: #https://github.com/elastic/response-ops-team/issues/209

This PR is a follow-on of #186804.

Creates a new task that runs every 1 hour to push the total connector-request-body-bytes that have been saved in the event log to usage-api.

@ersin-erdal ersin-erdal added release_note:skip Skip the PR/issue when compiling release notes Team:ResponseOps Label for the ResponseOps team (formerly the Cases and Alerting teams) backport:prev-minor Backport to (8.x) the previous minor version (i.e. one version back from main) ci:cloud-deploy Create or update a Cloud deployment labels Sep 30, 2024
@ersin-erdal ersin-erdal self-assigned this Sep 30, 2024
@elasticmachine
Copy link
Contributor

elasticmachine commented Sep 30, 2024

🤖 Jobs for this PR can be triggered through checkboxes. 🚧

ℹ️ To trigger the CI, please tick the checkbox below 👇

  • Click to trigger kibana-pull-request for this PR!

@ersin-erdal ersin-erdal added ci:cloud-redeploy Always create a new Cloud deployment and removed ci:cloud-deploy Create or update a Cloud deployment labels Sep 30, 2024
@kibana-ci
Copy link
Collaborator

kibana-ci commented Sep 30, 2024

💛 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

cc @ersin-erdal

@ersin-erdal ersin-erdal requested review from Zacqary and removed request for Zacqary October 15, 2024 16:18
@ersin-erdal ersin-erdal added ci:project-deploy-elasticsearch Create an Elasticsearch Serverless project and removed ci:cloud-redeploy Always create a new Cloud deployment labels Oct 17, 2024
@@ -72,6 +72,8 @@ const connectorTypeSchema = schema.object({
maxAttempts: schema.maybe(schema.number({ min: MIN_MAX_ATTEMPTS, max: MAX_MAX_ATTEMPTS })),
});

export const DEFAULT_USAGE_API_URL = 'https://usage-api.elastic-system/api/v1/usage';
Copy link
Member

Choose a reason for hiding this comment

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

We can't do this, as this way of hard-coding the URL broke, when the URL changed :-)

We're in the process of figuring out what to do with these, but we likely need a central location (core? new plugin?) to maintain these things, including the ca certs that we've already added.

@ersin-erdal ersin-erdal added ci:project-deploy-observability Create an Observability project and removed ci:project-deploy-elasticsearch Create an Elasticsearch Serverless project labels Oct 17, 2024
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.
  • run docs-build : Re-trigger the docs validation. (use unformatted text in the comment!)

@elasticmachine
Copy link
Contributor

elasticmachine commented Oct 19, 2024

💚 Build Succeeded

  • Buildkite Build
  • Commit: 79e9ecc
  • Kibana Serverless Image: docker.elastic.co/kibana-ci/kibana-serverless:pr-194429-79e9ecc69138

Metrics [docs]

✅ unchanged

History

cc @ersin-erdal

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) ci:project-deploy-observability Create an Observability project release_note:skip Skip the PR/issue when compiling release notes Team:ResponseOps Label for the ResponseOps team (formerly the Cases and Alerting teams)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants