Skip to content

Added S3 gateway endpoint, STS and Kinesis interface endpoints#28

Open
gergelj wants to merge 1 commit intomainfrom
gergeljkis/aws-byovpc-vpc-endpoints
Open

Added S3 gateway endpoint, STS and Kinesis interface endpoints#28
gergelj wants to merge 1 commit intomainfrom
gergeljkis/aws-byovpc-vpc-endpoints

Conversation

@gergelj
Copy link
Contributor

@gergelj gergelj commented Mar 13, 2026

Pull Request

Description

Brief description of what this PR does and why.

Category

  • core-platform
  • data-engineering
  • data-governance
  • data-warehousing
  • genai-ml
  • launch-accelerator
  • workspace-setup

Type of Change

  • New project
  • Bug fix
  • Enhancement
  • Documentation

Project Details

Project Name: AWS BYOVPC
Purpose: Terraform code for workspace deployment
Technologies Used: Terraform

Testing

  • Code runs without errors
  • Documentation is complete
  • Used only synthetic data

Security Compliance ✅

  • No customer data, PII, or proprietary information
  • No credentials or access tokens
  • Only synthetic data used
  • Third-party licenses acknowledged

By submitting this PR, I confirm I have followed the CONTRIBUTING.md guidelines and security requirements.

@gergelj
Copy link
Contributor Author

gergelj commented Mar 13, 2026

This setup requires a separate subnet that holds the STS and Kinesis interface endpoints. This subnet is called 'intra' subnet.

@gergelj
Copy link
Contributor Author

gergelj commented Mar 13, 2026

Tests done:

  • cluster startup is successful
  • S3 storage is accessible (read/write)
  • route table is updated, so that the EC2s can access the S3 gateway endpoint
Screenshot 2026-03-13 at 17 53 33
  • prefix list ID (which resolves to com.amazonaws.<region>.s3) routes to the S3 gateway endpoint, which is more specific than the 0.0.0.0/0 that routes to the NAT gateway
Screenshot 2026-03-13 at 17 53 47

Copy link
Contributor

@jpgianfaldoni jpgianfaldoni left a comment

Choose a reason for hiding this comment

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

LGTM

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.

2 participants