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

[Docs Fix] Update links to sample generated outputs from built-in conf #1835

Closed
bradleyhrc opened this issue Nov 25, 2024 · 3 comments · Fixed by #1836
Closed

[Docs Fix] Update links to sample generated outputs from built-in conf #1835

bradleyhrc opened this issue Nov 25, 2024 · 3 comments · Fixed by #1836
Labels
requires-doc Improvements or additions to markdown files (documentation)

Comments

@bradleyhrc
Copy link
Contributor

bradleyhrc commented Nov 25, 2024

Describe the bug
The linked samples in the Test config generation manually section of the dev-docs are broken. They should be updated to what I believe are these new paths:

@bradleyhrc bradleyhrc added the requires-doc Improvements or additions to markdown files (documentation) label Nov 25, 2024
@braydonk
Copy link
Contributor

I think the right output for otel is otel.yaml. The other two look right. Do they not match what happens when running the instructions?

@bradleyhrc
Copy link
Contributor Author

Yeah you're right it's otel.yaml -- I forgot to update the text. The link in it points to otel.yaml. The results match, just that the respective links are currently broken in the current dev doc.

@braydonk
Copy link
Contributor

Oh right I understand what the issue is now. Yeah, those links are to the old format for our testdata folder, which has since changed. Those updated URLs seem right to me.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
requires-doc Improvements or additions to markdown files (documentation)
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants