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

github-action: delete opentelemetry workflow #39559

Merged
merged 1 commit into from
May 15, 2024

Conversation

v1v
Copy link
Member

@v1v v1v commented May 14, 2024

Details

⚠️ This PR was created by an automated tool. Please review the changes carefully. ⚠️

We want to delete the opentelemetry workflow.

Why

We now support Observing the CICD at large scale without the need to use a specialised
workflow.

Further details

https://ela.st/oblt-ci-cd-stats is the place to see the traces and logs for
all the GitHub actions.

There will be one service per GitHub repository, including the org name, and
one Transaction per Workflow.

If there are any questions, please reach out to the @elastic/observablt-ci

@v1v v1v requested a review from a team as a code owner May 14, 2024 13:54
@v1v v1v self-assigned this May 14, 2024
@v1v v1v requested a review from a team May 14, 2024 13:54
@botelastic botelastic bot added the needs_team Indicates that the issue/PR needs a Team:* label label May 14, 2024
@botelastic
Copy link

botelastic bot commented May 14, 2024

This pull request doesn't have a Team:<team> label.

Copy link
Contributor

mergify bot commented May 14, 2024

This pull request does not have a backport label.
If this is a bug or security fix, could you label this PR @v1v? 🙏.
For such, you'll need to label your PR with:

  • The upcoming major version of the Elastic Stack
  • The upcoming minor version of the Elastic Stack (if you're not pushing a breaking change)

To fixup this pull request, you need to add the backport labels for the needed
branches, such as:

  • backport-v8./d.0 is the label to automatically backport to the 8./d branch. /d is the digit

@v1v v1v enabled auto-merge (squash) May 14, 2024 15:52
Copy link
Contributor

@dliappis dliappis left a comment

Choose a reason for hiding this comment

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

LGTM

@v1v v1v merged commit 8b58916 into main May 15, 2024
8 of 9 checks passed
@v1v v1v deleted the gh-oblt/delete-opentelemetry-workflow branch May 15, 2024 12:44
v1v added a commit to v1v/beats that referenced this pull request May 15, 2024
…-actions

* upstream/main: (313 commits)
  github-action: delete opentelemetry workflow (elastic#39559)
  updatecli: move to the .github folder and support for signed commits (elastic#39472)
  Osquerybeat: Add action responses data stream (elastic#39143)
  [winlogbeat] performance improvment; avoid rendering event message twice (elastic#39544)
  Fix the AWS SDK dependencies issue causing the "not found, ResolveEndpointV2" error (elastic#39454)
  x-pack/filebeat/input/cel: add http metrics collection (elastic#39503)
  build(deps): bump github.com/elastic/elastic-agent-libs from 0.9.4 to 0.9.7 (elastic#39424)
  Remove unused env vars from pipelines (elastic#39534)
  [BK] - Remove osx steps from branch execution (elastic#39552)
  [BK] - Remove certain steps from running for Branches (elastic#39533)
  Allow dependabot report BK status checks (elastic#39540)
  Remove hardcoded module definitions in CI (elastic#39506)
  Explicitly set DOCKER_PULL, RACE_DETECTOR and TEST_COVERAGE for pipelines (elastic#39510)
  Fixed pipelines formatting (elastic#39513)
  Update filebeat pipeline to match Jenkins steps (elastic#39261)
  Add error check to groupToEvents so we don't blindly add error values (elastic#39404)
  Remove fields not needed for session view in add_session_view processor (elastic#39500)
  `aws-s3` input: Split S3 poller and SQS reader into explicit input objects (elastic#39353)
  ci(jenkins): remove post-build notifications (elastic#39483)
  [DOCS] Add the `read_pipeline` cluster privilege for winlogbeat and the `auto_configure` index privilege to beats documentation (elastic#38534)
  ...
@v1v
Copy link
Member Author

v1v commented May 23, 2024

@Mergifyio backport 7.17

Copy link
Contributor

mergify bot commented May 23, 2024

backport 7.17

✅ Backports have been created

mergify bot pushed a commit that referenced this pull request May 23, 2024
(cherry picked from commit 8b58916)

# Conflicts:
#	.github/workflows/opentelemetry.yml
@v1v
Copy link
Member Author

v1v commented May 23, 2024

I'm gonna backport to 7.17 even if those actions are not used... but I think it can help with avoiding misleading... for 8.x branches is not needed, eventually 8.14 will not be maintained and 8.15 won't contain those changes

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs_team Indicates that the issue/PR needs a Team:* label
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants