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

📐 Add ADR proposals #3107

Merged
merged 35 commits into from
Feb 15, 2024
Merged

📐 Add ADR proposals #3107

merged 35 commits into from
Feb 15, 2024

Conversation

bagg3rs
Copy link
Contributor

@bagg3rs bagg3rs commented Jan 27, 2024 •
edited
Loading

This pull request adds proposals the below items, please add corrections or push changes where you think its needed.

  • AWS Bedrock, a managed service for large language models. Bedrock provides pre-trained models that can be customized and fine-tuned for various use cases. It offers serverless access to these models, eliminating the need for managing infrastructure.
  • AWS QuickSight for an alternative visualisation tool to RShiny
  • Using separate AWS Data Accounts to secure our data workloads
  • AWS SageMaker for analytical tooling
  • Documentation - Where do we store things

The pull request also includes other miscellaneous updates and formatting changes.

#2998

@bagg3rs
📝 Change 000 to use page page.title
6421207 
if using as template add page title
@bagg3rs
Add ADR-007 and ADR-008 for consideration
0acbd58 
Signed-off-by: Richard Baguley <20583399+bagg3rs@users.noreply.github.com>
@bagg3rs
Update QuickSight documentation and add AWS Bedrock for language mode…
ad1f39e 
…ls, documentation and using separate aws accounts drafts

Signed-off-by: Richard Baguley <20583399+bagg3rs@users.noreply.github.com>
@bagg3rs
Update documentation and use separate AWS accounts for data storage
f6ea2c7 
Signed-off-by: Richard Baguley <20583399+bagg3rs@users.noreply.github.com>
@bagg3rs
Update formatting
bbde7f5 
Signed-off-by: Richard Baguley <20583399+bagg3rs@users.noreply.github.com>
@bagg3rs
Add SCP and OU info
cc31be8 
Signed-off-by: Richard Baguley <20583399+bagg3rs@users.noreply.github.com>
@bagg3rs
Refactor AWS account structure for improved governance and security
0c7330d 
Signed-off-by: Richard Baguley <20583399+bagg3rs@users.noreply.github.com>
@bagg3rs
Update vendor or partner access in ADR-011
78dc87d 
Signed-off-by: Richard Baguley <20583399+bagg3rs@users.noreply.github.com>
@bagg3rs bagg3rs linked an issue Jan 27, 2024 that may be closed by this pull request
📄 Add ADR's from the DPAT priorities Miro #2998
Closed
2 tasks
@bagg3rs bagg3rs changed the title Update documentation and add AWS Bedrock for language models 📐 Add proposal ADR's Jan 27, 2024
@bagg3rs
Merge branch 'main' into add-adrs
8780929
@bagg3rs
📝 ⬆️ ⬇️ move and 🔥 words
62108e5 
Signed-off-by: Richard Baguley <20583399+bagg3rs@users.noreply.github.com>
@bagg3rs
🔥 remove temp file
3d37b42 
Signed-off-by: Richard Baguley <20583399+bagg3rs@users.noreply.github.com>
@bagg3rs
📝 clear up context
8f0027b 
Signed-off-by: Richard Baguley <20583399+bagg3rs@users.noreply.github.com>
@bagg3rs
📝 tidy up
7b05da8 
Signed-off-by: Richard Baguley <20583399+bagg3rs@users.noreply.github.com>
@bagg3rs bagg3rs changed the title 📐 Add proposal ADR's 📐 Add ADR proposals Jan 27, 2024
@bagg3rs
🔄 accept bedrock 🤔 and formatting and clarity for quicksight
74e918d 
Signed-off-by: Richard Baguley <20583399+bagg3rs@users.noreply.github.com>
@bagg3rs
🔄 clean up the mess a little more
d03f740 
Signed-off-by: Richard Baguley <20583399+bagg3rs@users.noreply.github.com>
@bagg3rs
spelling
eaffe9e 
Signed-off-by: Richard Baguley <20583399+bagg3rs@users.noreply.github.com>
@bagg3rs
Update ADR-008 AWS Bedrock documentation
2516b30 
Signed-off-by: Richard Baguley <20583399+bagg3rs@users.noreply.github.com>
@bagg3rs bagg3rs mentioned this pull request Jan 29, 2024
Closed
4 tasks
@bagg3rs
fix linter issues
f7c052b 
Signed-off-by: Richard Baguley <20583399+bagg3rs@users.noreply.github.com>
@bagg3rs
Bedrock status ✅ -> 🤔
fcd9477 
Signed-off-by: Richard Baguley <20583399+bagg3rs@users.noreply.github.com>
@bagg3rs
Update ADR-009: Use AWS SageMaker for analytical tooling
9744ea6 
Signed-off-by: Richard Baguley <20583399+bagg3rs@users.noreply.github.com>
@bagg3rs
Update ADR-009 to include benefits of using AWS SageMaker for analyti…
218af22 
…cal tooling

Signed-off-by: Richard Baguley <20583399+bagg3rs@users.noreply.github.com>
@bagg3rs
Update tooling from EKS to AWS SageMaker for improved efficiency and …
83edb30 
…reduced maintenance

Signed-off-by: Richard Baguley <20583399+bagg3rs@users.noreply.github.com>
@bagg3rs
Update ADR-009 remove duplicate consequences
7488003 
Signed-off-by: Richard Baguley <20583399+bagg3rs@users.noreply.github.com>
@bagg3rs
clarify
06d5a5c 
Signed-off-by: Richard Baguley <20583399+bagg3rs@users.noreply.github.com>
@bagg3rs
⬆️ update review date
07ce3d8 
Signed-off-by: Richard Baguley <20583399+bagg3rs@users.noreply.github.com>
@bagg3rs
✅ accepted identity and updated consequences
b5c5bc9 
Signed-off-by: Richard Baguley <20583399+bagg3rs@users.noreply.github.com>
@bagg3rs
spelling
6e024ee 
Signed-off-by: Richard Baguley <20583399+bagg3rs@users.noreply.github.com>
@bagg3rs
update review dates and rename Azure to EntraID
07f9607 
Signed-off-by: Richard Baguley <20583399+bagg3rs@users.noreply.github.com>
@bagg3rs
🚨 fix space
ba51c22 
Signed-off-by: Richard Baguley <20583399+bagg3rs@users.noreply.github.com>
@bagg3rs
📝 Review and Update ADR-007 @julialawrence
a922406 
Signed-off-by: Richard Baguley <20583399+bagg3rs@users.noreply.github.com>
@bagg3rs bagg3rs mentioned this pull request Feb 13, 2024
Closed
5 tasks
@bagg3rs
📝 Update ADR-009 to use AWS SageMaker for analytical tooling
ad67304 
Signed-off-by: Richard Baguley <20583399+bagg3rs@users.noreply.github.com>
@bagg3rs
Update ADR-008: Add information about Amazon Bedrock
09442f5 
Signed-off-by: Richard Baguley <20583399+bagg3rs@users.noreply.github.com>
@bagg3rs
Add data sovereignty issue note to ADR-008 AWS Bedrock
52f2b74 
Signed-off-by: Richard Baguley <20583399+bagg3rs@users.noreply.github.com>
@bagg3rs
Update ADR-009: Use AWS SageMaker for analytical tooling
68bdeef 
Signed-off-by: Richard Baguley <20583399+bagg3rs@users.noreply.github.com>
julialawrence
julialawrence approved these changes Feb 15, 2024
View reviewed changes
Copy link
Contributor

@julialawrence julialawrence left a comment

Choose a reason for hiding this comment

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

I agree with the contents but not with the separate account approach! Making that clear!

@bagg3rs bagg3rs merged commit 9540aac into main Feb 15, 2024
11 checks passed
@bagg3rs bagg3rs deleted the add-adrs branch February 15, 2024 12:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@julialawrence julialawrence julialawrence approved these changes

Assignees

@bagg3rs bagg3rs

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

📄 Add ADR's from the DPAT priorities Miro
2 participants
@bagg3rs @julialawrence