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

Add S3 Access Points and S3 Object Lambda Access Points icons #1083

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

kakakakakku
Copy link
Contributor

Hi 😀

Recently, I thought about trying create architecture diagrams as like this on Amazon S3 Object Lambda document, but I couldn't find it. The motivation for this pull request is to add the Amazon S3 Access Points icon and the Amazon S3 Object Lambda Access Points icon.

image

Check ✔️

I checked with my sample code.

if __name__ == "__main__":
    from diagrams import Diagram, Edge
    from diagrams.aws.general import Client
    from diagrams.aws.compute import LambdaFunction
    from diagrams.aws.storage import S3AccessPoints, S3ObjectLambdaAccessPoints, SimpleStorageServiceS3

    with Diagram('', filename='./sample', show=False):
        client = Client('Client')
        object_lambda = S3ObjectLambdaAccessPoints('S3ObjectLambdaAccessPoints')
        function = LambdaFunction('LambdaFunction')
        access_points = S3AccessPoints('S3AccessPoints')
        bucket = SimpleStorageServiceS3('SimpleStorageServiceS3')

        client >> Edge(label='API Request') >> object_lambda
        object_lambda >> Edge('Transformed Request') << function
        function >> Edge('Transformed Request') << access_points
        access_points >> Edge('API Request') << bucket

image

Could you please review? Thanks a lot 👍

Copy link
Collaborator

@tvqphuoc01 tvqphuoc01 left a comment

Choose a reason for hiding this comment

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

LGTM thank you @kakakakakku 👍

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