[BUG] Action does not fail in case of missing image about mongodb-github-action

[marcuspoehls]

@Segelzwerg Hey Marcel, the mongo:6.0 image should be available in the Docker registry. We’re testing this action against MongoDB 6.0 as well and it works fine.

The error you’re pointing out comes from docker. This action doesn’t catch Docker issues, yet. Is your pipeline in GitHub Actions (your linked pipeline) downloading Docker images when not finding them locally?

[Segelzwerg]

@Segelzwerg Hey Marcel, the mongo:6.0 image should be available in the Docker registry. We’re testing this action against MongoDB 6.0 as well and it works fine.

The error you’re pointing out comes from docker. This action doesn’t catch Docker issues, yet. Is your pipeline in GitHub Actions (your linked pipeline) downloading Docker images when not finding them locally?

@marcuspoehls thanks for the quick reply, Marcus, The CI is working, but I was merely pointing out that when the actions tries to retrieve an image, which doesn't exist it should rather fail instead be marked as completed. I hope described it better this time. Let me know if you require further information.

[marcuspoehls]

Ah, I see. Do you know how this action can check if Docker fails to pull a non-existent image? I appreciate a pull request if you can help me out here :)

[Segelzwerg]

Ah, I see. Do you know how this action can check if Docker fails to pull a non-existent image? I appreciate a pull request if you can help me out here :)

Currently, I don't know, but maybe I can look into it. However, I can't give you an estimate of how long it might take.

[marcuspoehls]

Unrelated: this action now supports MongoDB 6.0 in version 1.8.0. It needed an update to use mongosh and #35 solved it.

[ecarruda]

Hey @marcuspoehls I reported on my PR mentioned to this issue, but I don't know if you are still receiving notifications because it is closed, so, maybe something happened on publishing it to the marketplace, can you check it and tell us if you want some update on README or if you are going to update version tag on Marked?

[Segelzwerg]

I think I found a way. I'll make PR over the weekend.

[ecarruda]

I think I found a way. I'll make PR over the weekend.

everything is working good now, what kind of issue are you still having @Segelzwerg ?

[ecarruda]

Taking a look on your actions @Segelzwerg i think that you only need to update your tag to the latest one 1.8.0 and that is all man :)

[ecarruda]

just wait for the follow up from @marcuspoehls to check what kind of change he is going to do because of the market tag that i mentioned before.

[Segelzwerg]

@ecarruda my issue is not related to 6.0 specifically, but when a user tries to pull a non existing image. Then action shall fail and not be marked as completed

[ecarruda]

ahh yes. sure. just get the docker status on the creation and exit with error

On Fri, 26 Aug 2022 at 12:17 Marcel ***@***.***> wrote: @ecarruda <https://github.com/ecarruda> my issue is not related to 6.0 specifically, but when a user tries to pull a non existing image. Then action shall fail and not be marked as completed — Reply to this email directly, view it on GitHub <#36 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAFSZOE322KRBNF7GL57AYDV3DNXXANCNFSM57WXP4NA> . You are receiving this because you were mentioned.Message ID: ***@***.***>

-- Enviado do Gmail para celular

[Segelzwerg]

ahh yes. sure. just get the docker status on the creation and exit with
error

On Fri, 26 Aug 2022 at 12:17 Marcel @.***> wrote:

@ecarruda https://github.com/ecarruda my issue is not related to 6.0
specifically, but when a user tries to pull a non existing image. Then
action shall fail and not be marked as completed

—
Reply to this email directly, view it on GitHub
#36 (comment),
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAFSZOE322KRBNF7GL57AYDV3DNXXANCNFSM57WXP4NA
.
You are receiving this because you were mentioned.Message ID:
@.***>

--
Enviado do Gmail para celular

Yes right. I tried local already and can make PR later if you want. But have an appointment now.