Wandalen / wretry.action

Retry action for Github CI
MIT License
98 stars 22 forks source link

Input does not meet YAML 1.2 "Core Schema #42

Closed PrimalPimmy closed 2 years ago

PrimalPimmy commented 2 years ago

https://github.com/PrimalPimmy/tremor-runtime/runs/6396863368?check_suite_focus=true

Getting this error when running the workflow. My inputs are:

        - name: Publish tremor-script to crates.io
          uses: Wandalen/wretry.action@v1.0.11
          with:
            action: katyo/publish-crates@v1
            with: |
              path: './tremor-script'
              args: --no-verify
              dry-run: true 
            attempt_limit: 6
            attempt_delay: 10000

I can't point out what's wrong here, seems like a configuration issue here

dmvict commented 2 years ago

Hello, @PrimalPimmy

Thank you for your issue. We'll investigate and solve this bug. Now, our command have a lot of work so, we'll solve bug a little later. Thank you for your patience.

ermirizio commented 2 years ago

Hi! Experiencing same error here while trying to retru on docker/buil-push-action@v3

      - name: retry docker build
        uses: Wandalen/wretry.action@v1.0.19
        with:
          action: docker/build-push-action@v3
          with: |
            context: .
            push: false
            tags: app:latest
          attempt_limit: 3
          attempt_delay: 2000

Can't share the logs since is a private repo, here is a picture:

image

dmvict commented 2 years ago

Hello, @PrimalPimmy and @ermirizio

I've fixed problem described in the issue. See pull request #46

The problem was in default action inputs. The PR should fix yours problem.

Notice, that wretry.action cannot setup defaults from Github environments like https://github.com/katyo/publish-crates/blob/main/action.yml#L10. It will be next step to improve action.

dmvict commented 2 years ago

@PrimalPimmy, @ermirizio

The new version released.

Please, confirm that your problem is solved and close the issue.