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.

Sign up for GitHub

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

Jump to bottom

[mdatagen] mark mdatagen in this repo deprecated #30173

Merged
merged 2 commits into from
Dec 22, 2023
Merged

[mdatagen] mark mdatagen in this repo deprecated #30173

merged 2 commits into from
Dec 22, 2023

Conversation

codeboten
Copy link
Contributor

It's being moved to core to be utilized in both repositories.

[mdatagen] mark mdatagen in this repo deprecated
a9ca607 
It's being moved to core to be utilized in both repositories.

Signed-off-by: Alex Boten <aboten@lightstep.com>
@codeboten codeboten requested a review from dmitryax as a code owner December 21, 2023 23:17
@codeboten codeboten requested a review from a team December 21, 2023 23:17
@github-actions github-actions bot added the cmd/mdatagen mdatagen command label Dec 21, 2023
add changelog
1395835 
Signed-off-by: Alex Boten <aboten@lightstep.com>
@bryan-aguilar
Copy link
Contributor 

 FILE: cmd/mdatagen/README.md
  [✖] https://github.com/open-telemetry/opentelemetry-collector/tree/main/cmd/mdatagen → Status: 404
  [✓] ./documentation.md → Status: 200
  [✓] https://github.com/open-telemetry/opentelemetry-collector/tree/main/receiver/otlpreceiver → Status: 200
  [✓] ./metadata-schema.yaml → Status: 200
  [✓] ../../receiver/elasticsearchreceiver/metadata.yaml → Status: 200
  [✓] ../../receiver/hostmetricsreceiver/internal/scraper/cpuscraper → Status: 200

  ERROR: 1 dead links found!
  [✓] ./loader.go → Status: 200
  [✓] ./templates → Status: 200
  [✓] metadata.yaml → Status: 200
  [✓] ./internal/metadata/generated_metrics_test.go → Status: 200

  10 links checked.
  [✖] https://github.com/open-telemetry/opentelemetry-collector/tree/main/cmd/mdatagen → Status: 404
Check that anchor links are lowercase

@codeboten
Copy link
Contributor Author

@bryan-aguilar yeah I prematurely added it before the other PR was merged, will wait to merge :)

@codeboten
Copy link
Contributor Author

Link has been fixed 👍🏻

@codeboten codeboten merged commit 1b0a44e into open-telemetry:main Dec 22, 2023
85 checks passed
@codeboten codeboten deleted the codeboten/deprecate-mdatagen branch December 22, 2023 18:21
@github-actions github-actions bot added this to the next release milestone Dec 22, 2023
cparkins pushed a commit to AmadeusITGroup/opentelemetry-collector-contrib that referenced this pull request Jan 10, 2024
@cparkins
[mdatagen] mark mdatagen in this repo deprecated (open-telemetry#30173)
aa03ff8 
It's being moved to core to be utilized in both repositories.

---------

Signed-off-by: Alex Boten <aboten@lightstep.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@crobert-1 crobert-1 crobert-1 approved these changes

@bryan-aguilar bryan-aguilar bryan-aguilar approved these changes

@songy23 songy23 songy23 approved these changes

@dmitryax dmitryax dmitryax approved these changes

@hughesjj hughesjj hughesjj approved these changes

@bogdandrutu bogdandrutu bogdandrutu approved these changes

Assignees

@bryan-aguilar bryan-aguilar

Labels
cmd/mdatagen mdatagen command
Projects
None yet
Milestone
v0.92.0
Development

Successfully merging this pull request may close these issues.
7 participants
@codeboten @bryan-aguilar @bogdandrutu @hughesjj @dmitryax @songy23 @crobert-1