Skip to content

Commit bac9024

Browse files
authored
Fix environment variables and inputs (#31)
1 parent e6899fa commit bac9024

File tree

5 files changed

+18
-18
lines changed

5 files changed

+18
-18
lines changed

README.md

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -283,7 +283,7 @@ jobs:
283283
```
284284

285285
## login_to_aks
286-
Get AKS login credentials for kubectl
286+
Get AKS login credentials for kubectl. Service principal used must have a permission to list cluster credentials.
287287

288288
### Inputs
289289
| Name | Description | Optional | Default value |
@@ -326,12 +326,12 @@ Copy python site-packages of current virtual environment and installs applicatio
326326
### Inputs
327327
| Name | Description | Optional | Default value |
328328
|-------------------|:-----------------------------------------------------------------|----------|---------------|
329-
| output_directory | Local directory on build agent to store files | False | |
330329
| project_version | Version of the project | False | |
331330
| project_name | Name of the project | False | |
332331
| project_directory | Directory name inside the project (if differs from project name) | True | "" |
333332
| destination | Directory or SAS for upload | False | |
334333
| python_version | Project python version | True | 3.9 |
334+
| deployment_root | Root directory in the file share | False | |
335335

336336
**NOTES**:
337337
1) To use this action, your project should use poetry for virtual environment management. Ensure that you installed
@@ -380,10 +380,10 @@ Prepare DBT models for deployment to an Azure file share.
380380
### Inputs
381381
| Name | Description | Optional | Default value |
382382
|------------------|:----------------------------------------------|----------|---------------|
383-
| output_directory | Local directory on build agent to store files | False | |
384383
| project_version | Version of the project | False | |
385384
| project_name | Name of the project | False | |
386385
| destination | Directory or SAS for upload | False | |
386+
| deployment_root | Root directory in the file share | False | |
387387

388388
### Outputs
389389
No outputs defined
@@ -428,10 +428,10 @@ Prepare DBT schemas for deployment to an Azure file share.
428428
### Inputs
429429
| Name | Description | Optional | Default value |
430430
|------------------|:----------------------------------------------|----------|---------------|
431-
| output_directory | Local directory on build agent to store files | False | |
432431
| project_version | Version of the project | False | |
433432
| project_name | Name of the project | False | |
434433
| destination | Directory or SAS for upload | False | |
434+
| deployment_root | Root directory in the file share | False | |
435435

436436
### Outputs
437437
No outputs defined

deploy_data_schemas_to_azfs/action.yaml

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -6,10 +6,6 @@ branding:
66
color: 'green'
77

88
inputs:
9-
output_directory:
10-
description: Output directory
11-
required: true
12-
139
project_version:
1410
description: Version of project to deploy
1511
required: true
@@ -22,6 +18,10 @@ inputs:
2218
required: true
2319
description: Destination directory or SAS token
2420

21+
deployment_root:
22+
required: true
23+
description: Root directory in the deployment file structure
24+
2525
runs:
2626
using: "composite"
2727
steps:

deploy_dbt_project_to_azfs/action.yaml

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -6,10 +6,6 @@ branding:
66
color: 'green'
77

88
inputs:
9-
output_directory:
10-
description: Output directory
11-
required: true
12-
139
project_version:
1410
description: Version of project to deploy
1511
required: true
@@ -22,6 +18,10 @@ inputs:
2218
required: true
2319
description: Destination directory or SAS token
2420

21+
deployment_root:
22+
required: true
23+
description: Root directory in the deployment file structure
24+
2525
runs:
2626
using: "composite"
2727
steps:

deploy_dbt_project_to_azfs/deploy_dbt_project_to_azfs.sh

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@
1717
set -Eeuo pipefail
1818

1919
SOURCE_DIRECTORY="./$DEPLOYMENT_ROOT/$PROJECT_NAME/$PROJECT_VERSION/"
20-
mkdir -p SOURCE_DIRECTORY
20+
mkdir -p "$SOURCE_DIRECTORY"
2121
mv -v ./target/run/* "$SOURCE_DIRECTORY"
2222

2323
./azcopy copy "./$SOURCE_DIRECTORY/*" "$DESTINATION" --recursive --overwrite true --put-md5

deploy_poetry_project_to_azfs/action.yaml

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -6,10 +6,6 @@ branding:
66
color: 'green'
77

88
inputs:
9-
output_directory:
10-
description: Output directory
11-
required: true
12-
139
project_version:
1410
description: Version of project to deploy
1511
required: true
@@ -31,14 +27,18 @@ inputs:
3127
required: false
3228
default: "3.9"
3329

30+
deployment_root:
31+
required: true
32+
description: Root directory in the deployment file structure
33+
3434
runs:
3535
using: "composite"
3636
steps:
3737
- name: Prepare python deployment
3838
shell: bash
3939
run: $GITHUB_ACTION_PATH/deploy_poetry_project_to_azfs.sh
4040
env:
41-
OUTPUT_DIRECTORY: ${{ inputs.output_directory }}
41+
DEPLOYMENT_ROOT: ${{ inputs.deployment_root }}
4242
PROJECT_VERSION: ${{ inputs.project_version }}
4343
PROJECT_NAME: ${{ inputs.project_name }}
4444
PROJECT_DIRECTORY : ${{ inputs.project_directory }}

0 commit comments

Comments
 (0)