Skip to content

Conversation

@eddie-knight
Copy link
Collaborator

This PR attempts to demonstrate what it could look like to draw elements from service-specific control catalogs into a control catalog for a reference architecture.

@netlify
Copy link

netlify bot commented May 18, 2025

Deploy Preview for common-cloud-controls canceled.

Name Link
🔨 Latest commit 265c335
🔍 Latest deploy log https://app.netlify.com/projects/common-cloud-controls/deploys/685b534b0ee8a70008f8c9bc

Signed-off-by: Eddie Knight <[email protected]>
@eddie-knight
Copy link
Collaborator Author

@mlysaght2017 What do you think of this approach?

Copy link
Contributor

@mlysaght2017 mlysaght2017 left a comment

Choose a reason for hiding this comment

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

Looks good!

@github-actions
Copy link

This issue will be closed as stale in 7 days. If this issue is blocked,
please tag or assign the appropriate party to move this forward.

@github-actions github-actions bot added the stale label Jun 18, 2025
@eddie-knight eddie-knight changed the title Creating catalog for 3-tier web app architecture add(arch): catalog for 3-tier web app architecture Jun 23, 2025
@eddie-knight
Copy link
Collaborator Author

I think this is in a reasonable state, ready for addition of the actual threats and controls.

If the team prefers, I can make the relevant ci changes first in a separate PR:

  • PR Title checker won't currently pass because this adds arch as a new option
  • yaml-checker won't pass because this modifies the expectations for metadata — all metadata files will need updated to comply with this schema

@eddie-knight eddie-knight marked this pull request as ready for review June 23, 2025 19:20
@eddie-knight eddie-knight requested review from a team as code owners June 23, 2025 19:20
@github-actions github-actions bot removed the stale label Jun 23, 2025
Signed-off-by: Eddie Knight <[email protected]>
Copy link
Contributor

@damienjburks damienjburks left a comment

Choose a reason for hiding this comment

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

This is so dope! Love it 🔥

@eddie-knight eddie-knight merged commit 5450942 into finos:main Jun 25, 2025
12 checks passed
@eddie-knight eddie-knight deleted the feat/3twa branch June 25, 2025 10:51
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.

3 participants