Skip to content

S3 Django Sample App Deployment workflow #6

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

Merged
merged 9 commits into from
Mar 1, 2024
Merged

Conversation

zzhlogin
Copy link
Contributor

@zzhlogin zzhlogin commented Mar 1, 2024

This PR create App Signals Enablement - S3 Django Sample App Deployment workflow to upload sample app ZIP file to all regions similar as App Signals Enablement - S3 Springboot Sample App Deployment.

The difference between Java is:
In python, we only need one zip file compressing two sample app micro-services instead of two.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Copy link
Contributor

@jj22ee jj22ee left a comment

Choose a reason for hiding this comment

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

LGTM. I assume the actual buckets do not exist yet?

@zzhlogin zzhlogin merged commit 6a8e54f into main Mar 1, 2024
zzhlogin added a commit to zzhlogin/aws-application-signals-test-framework that referenced this pull request Jun 6, 2024
…deploy_new

S3 Django Sample App Deployment workflow
zzhlogin added a commit to zzhlogin/aws-application-signals-test-framework that referenced this pull request Jun 6, 2024
…deploy_new

S3 Django Sample App Deployment workflow
zzhlogin added a commit to zzhlogin/aws-application-signals-test-framework that referenced this pull request Jun 6, 2024
…deploy_new

S3 Django Sample App Deployment workflow
zzhlogin added a commit to zzhlogin/aws-application-signals-test-framework that referenced this pull request Jun 6, 2024
…deploy_new

S3 Django Sample App Deployment workflow
georgeboc pushed a commit to georgeboc/aws-application-signals-test-framework that referenced this pull request Jul 8, 2024
*Description of changes:*

In this commit, we are adding and enabling a checker that ensures a
copywrite header is added to every code file. Also we are fixing a
warning message by updating the .pylintrc to reference
"builtins.Exception" instead of "Exception". Also we apply header to
files missing it.

By submitting this pull request, I confirm that you can use, modify,
copy, and redistribute this contribution, under the terms of your
choice.
zzhlogin added a commit that referenced this pull request Sep 13, 2024
# This is the 1st commit message:

Test concurency.

# The commit message #2 will be skipped:

# Dummy commit

# The commit message #3 will be skipped:

# Dummy commit

# The commit message #4 will be skipped:

# Dummy commit

# The commit message #5 will be skipped:

# Dummy commit

# The commit message #6 will be skipped:

# add push.

# The commit message #7 will be skipped:

# Dummy commit

# The commit message #8 will be skipped:

# Dummy commit

# The commit message #9 will be skipped:

# Dummy commit

# The commit message #10 will be skipped:

# Add push condition.

# The commit message #11 will be skipped:

# Dummy commit
zzhlogin added a commit that referenced this pull request Sep 17, 2024
# This is the 1st commit message:

Test concurency.

# The commit message #2 will be skipped:

# Dummy commit

# The commit message #3 will be skipped:

# Dummy commit

# The commit message #4 will be skipped:

# Dummy commit

# The commit message #5 will be skipped:

# Dummy commit

# The commit message #6 will be skipped:

# add push.

# The commit message #7 will be skipped:

# Dummy commit

# The commit message #8 will be skipped:

# Dummy commit

# The commit message #9 will be skipped:

# Dummy commit

# The commit message #10 will be skipped:

# Add push condition.

# The commit message #11 will be skipped:

# Dummy commit
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