Skip to content
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

GitHub Action - service "cli" is not running container #1 #380

Closed
ynnoig opened this issue Oct 14, 2022 · 7 comments
Closed

GitHub Action - service "cli" is not running container #1 #380

ynnoig opened this issue Oct 14, 2022 · 7 comments
Labels
bug Something isn't working solved

Comments

@ynnoig
Copy link

ynnoig commented Oct 14, 2022

Which release, branch or hash of Docker SDK are you using?

1.49.4

Which operating system (platform/version/architecture) are you using?

Linux Ubuntu on GitHub Actions

What is the issue that you're experiencing?

Sometimes in ci jobs that use docker-sdk we have the following error:
service "cli" is not running container #1 /home/runner/work/project/project/docker/deployment/default/bin/service/database/postgres.sh: line 49: [: : integer expression expected

What are the steps to reproduce the issue?

  php-81-postgresql-integration-alpine:
    name: "PHP 8.1 / PostgreSQL / Integration / Alpine"
    runs-on: ubuntu-latest
    env:
      ...

    steps:
    ...

      - name: Run docker
        run: |
          git clone <docker-sdk latest>
          docker/sdk boot -v deploy.ci.integration.yml
          docker/sdk up -t << Here happened the error!
          docker/sdk testing console queue:worker:start --stop-when-empty
          docker/sdk testing codecept run -c codeception.ci.integration.yml

What is the expected result of these steps in the absence of the issue?

That docker sdk start

What is the actual result?

Github Workflow stopped with exit code 1

What possible solutions and/or workarounds for the issue do you see?

I don't know

Is there any other information that might be helpful?

I think not

@alexanderM91
Copy link
Collaborator

Hey @ynnoig,

Could you please help us to understand the root cause and answer for several questions?

  • Do you have the same behaviour locally or does it show up only in CI?
  • Does CI stop execution or continue execution?
  • Did you have this problem with the previous release?

@ynnoig
Copy link
Author

ynnoig commented Oct 14, 2022

Hey @ynnoig,

Could you please help us to understand the root cause and answer for several questions?

  • Do you have the same behaviour locally or does it show up only in CI?
  • Does CI stop execution or continue execution?
  • Did you have this problem with the previous release?
  • Only in CI (GitHub Actions)
  • Stop Execution, I think due bash error in postgres.sh
  • Yes, but always sometime (i.e. one or two times per week)

Thank u for your reply!

@alexanderM91
Copy link
Collaborator

Cool, thanks a lot, @ynnoig, for your accurate and meaningful response.
We will put it into our backlog and mark as a bug with normal priority.

cc: @alexfoxpetrov

@alexanderM91
Copy link
Collaborator

Hi @ynnoig,
do you have any updates here?
Also, did you have a chance to reproduce it locally?
Unfortunately, without a proper test case we cannot reproduce and fix it as our attempts were unsuccessful.

@ynnoig
Copy link
Author

ynnoig commented Oct 26, 2022

Hi @alexanderM91

with the fix (#381) from @kalaputsik it works better... We had only on monday again a case with this error but not clear why. @kalaputsik has been notified from me about this case via Slack.

Locally runs always, only on GH Actions there was the problem.

I think that the optimization from @kalaputsik can be merged.

@alexanderM91
Copy link
Collaborator

Hi @alexanderM91

with the fix (#381) from @kalaputsik it works better... We had only on monday again a case with this error but not clear why. @kalaputsik has been notified from me about this case via Slack.

Locally runs always, only on GH Actions there was the problem.

I think that the optimization from @kalaputsik can be merged.

Thank you, @ynnoig, for your reply.
Awesome, we will proceed with our internal testing.

@alexanderM91
Copy link
Collaborator

Hi, the changes have been merged into the latest master.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working solved
Development

No branches or pull requests

2 participants