teamdable / vv-deploy-actions

1 stars 0 forks source link

수동 배포시에 배포대상을 inputs로 받기 #22

Closed hyeseung-of-dable closed 2 years ago

hyeseung-of-dable commented 2 years ago

v1.0.3에서 추가 예정

python3 action/get-vpn-ip --include-api-live-inventory --allow-devices ${{ inputs.allow-devices }} --exclude-devices ${{ inputs.exclude-devices }} --exclude-inventory-id ${{ inputs.exclude-inventory-id }} 로 변경 고민중

hyeseung-of-dable commented 2 years ago
  1. 수동 배포시에 inputs로 받고, 비어있는 칸은 config파일을 그대로 받고싶음 -> 무조건 다 입력으로 대체해야함

  2. 수동 트리거 인풋에 디폴트값 주기 ->

      allow-devices:
        required: false
        default: '[]'

    이런 식으로 하면 됨

  3. include-api-live-inventory include-api-live-inventory=False 하려면 어떻게 해야하는지 . .

hyeseung-of-dable commented 2 years ago

TO-BE)

! usage: get-vpn-ip [-h] [-c CONFIG] [--include-api-live-inventory BOOLEAN]
                  [--allow-devices ALLOW_DEVICES]
                  [--exclude-devices EXCLUDE_DEVICES]
                  [--exclude-inventory-id EXCLUDE_INVENTORY_ID]
get-vpn-ip: error: argument --exclude-devices: expected one argument

invlude-api-live-inventory를 t/f값을 받도록 수정

hyeseung-of-dable commented 2 years ago

윤승님께서 해결완!