Skip to content

Conversation

@joelmccoy
Copy link
Contributor

@joelmccoy joelmccoy commented Oct 2, 2025

Description

Adds docs/best practices/architecture diagram for the UDS Core's Observability stack.

Related Issue

Fixes #1878

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Other (security config, docs update, etc)

Steps to Validate

  • uds run dev-docs and check the docs site locally (including embedded likec4 diagram)

Checklist before merging

@joelmccoy joelmccoy requested a review from Copilot October 2, 2025 18:49
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@joelmccoy joelmccoy mentioned this pull request Oct 8, 2025
5 tasks
@joelmccoy
Copy link
Contributor Author

Waiting until #2000 is merged in to build the observability overview diagram

@joelmccoy joelmccoy requested a review from Copilot October 27, 2025 18:19
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

Copilot reviewed 6 out of 6 changed files in this pull request and generated 1 comment.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Contributor

@mjnagel mjnagel left a comment

Choose a reason for hiding this comment

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

Looks great overall - lots of great information and a helpful overview + specific guides on each component. LGTM pending the diagram (or we can remove that TODO and ship as-is).

@joelmccoy joelmccoy marked this pull request as ready for review November 4, 2025 17:54
@joelmccoy joelmccoy requested a review from a team as a code owner November 4, 2025 17:54
@joelmccoy joelmccoy requested a review from Copilot November 4, 2025 17:55
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

Copilot reviewed 10 out of 12 changed files in this pull request and generated 1 comment.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

chance-coleman
chance-coleman previously approved these changes Nov 4, 2025
Copy link
Contributor

@mjnagel mjnagel left a comment

Choose a reason for hiding this comment

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

Few small diagram comments.

@joelmccoy joelmccoy merged commit afdac07 into main Nov 5, 2025
30 checks passed
@joelmccoy joelmccoy deleted the observability-arch branch November 5, 2025 20:18
mjnagel pushed a commit that referenced this pull request Nov 12, 2025
🤖 I have created a release *beep* *boop*
---


##
[0.56.0](v0.55.1...v0.56.0)
(2025-11-12)


### ⚠ BREAKING CHANGES

* remove neuvector and require falco
([#2083](#2083))

### Features

* conditional access request notes
([#2116](#2116))
([73cef02](73cef02))
* remove neuvector and require falco
([#2083](#2083))
([e43bcc3](e43bcc3))


### Bug Fixes

* **docs:** broken link in sso overview
([#2114](#2114))
([6738d42](6738d42))
* install test apps preupgrade and validate
([#2088](#2088))
([ed1211a](ed1211a))
* prevent network policies from selecting non-authservice SSO clients
([#2127](#2127))
([b95d997](b95d997))


### Miscellaneous

* add lula compliance mapping
([#2049](#2049))
([23d0163](23d0163))
* add lula mappings
([#2119](#2119))
([b8a5811](b8a5811))
* add lula mappings for controls
([#2102](#2102))
([3855b53](3855b53))
* add observability reference architecture docs
([#1987](#1987))
([afdac07](afdac07))
* cleanup unused oscal/lula1 tasks
([#2122](#2122))
([fd5c767](fd5c767))
* **deps:** update keycloak
([#2124](#2124))
([6a2097b](6a2097b))
* **deps:** update keycloak to v26.4.2
([#2061](#2061))
([5dd5cf8](5dd5cf8))
* **deps:** update keycloak to v26.4.4
([#2130](#2130))
([f0563a6](f0563a6))
* **deps:** update pepr to v1
([#2118](#2118))
([8cf4543](8cf4543))
* **deps:** update support-deps
([#2128](#2128))
([ba26eed](ba26eed))
* **docs:** add doc warning around remoteHost egress
([#2104](#2104))
([3d9bce5](3d9bce5))
* **docs:** remove neuvector docs
([#2095](#2095))
([29e92a1](29e92a1))
* **docs:** update diagrams
([#2109](#2109))
([ee2a67d](ee2a67d))
* **docs:** update likec4 diagram
([#2108](#2108))
([b90df25](b90df25))
* fix spelling in name of file
([#2112](#2112))
([bfe818c](bfe818c))
* fix test task to check authservice protection
([#2121](#2121))
([81d4d34](81d4d34))
* lula annotations
([#2074](#2074))
([03d497d](03d497d))
* mappings
([#2123](#2123))
([91d3756](91d3756))
* update to uds-k3d 0.19.1, remove uds-dev-stack exemption
([#2098](#2098))
([1f33957](1f33957))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
mjnagel pushed a commit to BagelLab/uds-core that referenced this pull request Nov 14, 2025
…#1987)

## Description

Adds docs/best practices/architecture diagram for the UDS Core's
Observability stack.

## Related Issue

Fixes defenseunicorns#1878 

## Type of change

- [ ] Bug fix (non-breaking change which fixes an issue)
- [ ] New feature (non-breaking change which adds functionality)
- [x] Other (security config, docs update, etc)

## Steps to Validate

- `uds run dev-docs` and check the docs site locally (including embedded
likec4 diagram)

## Checklist before merging

- [x] Test, docs, adr added or updated as needed
- [x] [Contributor
Guide](https://github.com/defenseunicorns/uds-template-capability/blob/main/CONTRIBUTING.md)
followed
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.

Establish/Document a UDS Core Observability Reference Architecture

4 participants