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

Update logging architecture #2315

Merged
merged 11 commits into from
Oct 24, 2024
Merged

Update logging architecture #2315

merged 11 commits into from
Oct 24, 2024

Conversation

TheoBrigitte
Copy link
Member

@TheoBrigitte TheoBrigitte commented Oct 21, 2024

Towards: giantswarm/roadmap#3518

This PR updates the logging architecture page to reflect the current state of the logging platform.

@TheoBrigitte TheoBrigitte requested a review from a team as a code owner October 21, 2024 09:37
@TheoBrigitte TheoBrigitte self-assigned this Oct 21, 2024
@TheoBrigitte TheoBrigitte requested a review from a team October 21, 2024 14:46
@pipo02mix
Copy link
Contributor

@TheoBrigitte can we point to the new docs? We are about to switch. This one will perfectly fit as a child of -> here

@TheoBrigitte
Copy link
Member Author

TheoBrigitte commented Oct 22, 2024

@pipo02mix What should point where ?

@pipo02mix
Copy link
Contributor

  * add principal menu

 * update relref references
@TheoBrigitte
Copy link
Member Author

Here you go I moved the logging architecture page under the new overview observability section.

image

Note that the old link in the vintage menu below will now redirect the user to the new menu

image

Copy link
Contributor

@pipo02mix pipo02mix left a comment

Choose a reason for hiding this comment

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

nice!

@QuentinBisson
Copy link
Contributor

Great work

@TheoBrigitte TheoBrigitte merged commit 04c927f into main Oct 24, 2024
8 checks passed
@TheoBrigitte TheoBrigitte deleted the logging-platform-update branch October 24, 2024 08:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants