Skip to content

Commit de1ffc8

Browse files
committed
VED-79: pipeline review
1 parent 33b2206 commit de1ffc8

File tree

3 files changed

+16
-16
lines changed

3 files changed

+16
-16
lines changed

azure/azure-release-pipeline.yml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -46,6 +46,7 @@ extends:
4646
- template: ./templates/post-deploy.yml
4747
parameters:
4848
aws_account_type: 'dev'
49+
subscribe_to_mns: false
4950
- environment: sandbox
5051
proxy_path: sandbox
5152
jinja_templates:

azure/templates/post-deploy.yml

Lines changed: 9 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -83,19 +83,15 @@ steps:
8383
8484
- ${{ if eq(parameters.subscribe_to_mns, true) }}:
8585
- bash: |
86-
export AWS_PROFILE=apim-dev
87-
echo "Subscribing SQS to MNS for notifications."
88-
pyenv install -s 3.11.11
89-
pyenv local 3.11.11
90-
echo "Setting up poetry environment..."
91-
poetry env use 3.11
92-
poetry install --no-root
93-
94-
echo "Setting PYTHONPATH..."
95-
export PYTHONPATH=$(Pipeline.Workspace)/s/$(SERVICE_NAME)/$(SERVICE_ARTIFACT_NAME)/mns_subscription
96-
97-
echo "Subscribing SQS to MNS for notifications..."
98-
poetry run python src/subscribe_mns.py
86+
export AWS_PROFILE=apim-dev
87+
echo "Subscribing SQS to MNS for notifications."
88+
pyenv install -s 3.11.11
89+
pyenv local 3.11.11
90+
echo "Setting up poetry environment..."
91+
poetry env use 3.11
92+
poetry install --no-root
93+
echo "Subscribing SQS to MNS for notifications..."
94+
poetry run python src/subscribe_mns.py
9995
displayName: "Run MNS Subscription"
10096
workingDirectory: "$(Pipeline.Workspace)/s/$(SERVICE_NAME)/$(SERVICE_ARTIFACT_NAME)/mns_subscription"
10197
env:

mns_subscription/Makefile

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,12 @@
11
test:
22
@PYTHONPATH=src:tests python -m unittest
33

4-
cover:
4+
coverage-run:
55
@PYTHONPATH=src:tests coverage run -m unittest discover
66

7-
report:
8-
coverage report -m
7+
coverage-report:
8+
coverage report -m
9+
10+
coverage-html:
11+
coverage html
912
.PHONY: build package test

0 commit comments

Comments
 (0)