Skip to content

Integration Test Workflow #246

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 53 commits into from
Nov 25, 2020
Merged

Integration Test Workflow #246

merged 53 commits into from
Nov 25, 2020

Conversation

bhautikpip
Copy link
Contributor

Issue #, if available:

Description of changes:

Integration test workflow with flask web app testing aws sdk, http, flask-sql alchemy instrumentation and metadata-annotations injection

  • Added terraform to deploy web app on elastic beanstalk
  • Added Github actions workflow to run end to end integration tests

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@bhautikpip bhautikpip changed the title Ci Integration Test Workflow Oct 12, 2020
@codecov-io
Copy link

codecov-io commented Oct 12, 2020

Codecov Report

Merging #246 (431b580) into master (2660af1) will increase coverage by 0.06%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #246      +/-   ##
==========================================
+ Coverage   79.29%   79.36%   +0.06%     
==========================================
  Files          80       80              
  Lines        3159     3159              
==========================================
+ Hits         2505     2507       +2     
+ Misses        654      652       -2     
Impacted Files Coverage Δ
aws_xray_sdk/core/sampling/local/reservoir.py 100.00% <0.00%> (+11.11%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2660af1...431b580. Read the comment docs.

@bhautikpip bhautikpip requested a review from anuraaga October 13, 2020 17:23
@bhautikpip
Copy link
Contributor Author

bhautikpip commented Oct 20, 2020

Action items:

  1. Work with new validator module (TraceValidator aws-observability/aws-otel-test-framework#32) - done
  2. Make sure main repo has aws account creds - done
  3. Put flask sample apps to sample apps repo
  4. Update the workflow to change repo download config, testing config

@bhautikpip bhautikpip merged commit 2b5a130 into aws:master Nov 25, 2020
Hargrav3s pushed a commit to Gavant/aws-xray-sdk-python that referenced this pull request Mar 22, 2022
* Adding integration testing workflow and terraform files

* Added workflow to execute on pull request only

* modified tfvars file name

* Added resource_prefix var, os name

* Applied constant var directly

* Added -var parameter to fetch github run_id env variable

* remove resource_prefix logic and solve secret repo access issue

* test checking out from secret repo

* test pulling secret repo

* fixing destroying resources

* added debug logic to fix plugins permissions issue

* debug logic to check plugins dir location

* removed download artifact step

* debug logic

* debug permission issue

* debug permission issue - 2

* debug permission issue

* debug permission issue

* debug destroy resources

* fix: destroy resources

* remove unnecessary steps

* fix: end to end workflow

* Added resource_prefix var

* Added env variable to set resource_prefix

* removed constant vars from tfvars and used env variable to populate values

* modified names

* fix: eb url

* minor changes

* minor changes

* Added sse encryption to s3 and eb updated config

* Added public access setting to bucket, sse encrpytion and fix eb issue

* eb issue: try

* Added integ test runner repo

* Added repo for sample app, test runner

* minor change

* execute workflow on push too

* Added secret PAT to test workflow

* Added validator module

* Added gradle execution command

* minor change

* minor change: added branch

* Added aws creds

* fixed issue

* minor cleanups

* modified testing module and webapp module

* Added logic to create unique names in case of re running same workflow

* added sample-apps dir

* fixed syntax issue

* added path in upload and download actions

* Added license file

* workflow executes on push
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