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

ironmq CI tests fail #697

Open
pjfanning opened this issue Jun 16, 2024 · 10 comments
Open

ironmq CI tests fail #697

pjfanning opened this issue Jun 16, 2024 · 10 comments

Comments

@pjfanning
Copy link
Contributor

pjfanning commented Jun 16, 2024

In all recent PR builds. - eg https://github.com/apache/pekko-connectors/actions/runs/9486627304

Creating network "pekko-connectors_default" with the default driver
Creating volume "pekko-connectors_kudu-tserver-data" with default driver
Pulling ironauth (iron/auth:)...
latest: Pulling from iron/auth
[DEPRECATION NOTICE] Docker Image Format v1 and Docker Image manifest version 2, schema 1 support is disabled by default and will be removed in an upcoming release. Suggest the author of docker.io/iron/auth:latest to upgrade the image to the OCI Format or Docker Image manifest v2, schema 2. More information at https://docs.docker.com/go/deprecated-image-specs/
Error: Process completed with exit code 1.
@laglangyue
Copy link
Contributor

refference to
etcd-io/etcd#17811
maybe we shoulde add a version for iron/auth and iron/mq
image

@pjfanning
Copy link
Contributor Author

https://hub.docker.com/r/iron/mq is unmaintained. No images have been published in years.

@samueleresca
Copy link
Member

https://hub.docker.com/r/iron/mq is unmaintained. No images have been published in years.
Yes. I think the error won't go away unless there is an image compliant with OCI standard / Docker v2.

We either:

  • Don't test on iron/mq
  • Republish a ironmq image that is compliant with OCI standard.
  • Reach out the ironmq team

@pjfanning
Copy link
Contributor Author

pjfanning commented Jun 16, 2024

I emailed IronMQ support earlier. I'll see if they respond.

@Roiocam
Copy link
Member

Roiocam commented Jun 16, 2024

refference to etcd-io/etcd#17811 maybe we shoulde add a version for iron/auth and iron/mq image

I think that's not quiet the same thing:

  • the refference issue is because etcd team didn't use v2 format to publish their latest tag.
  • this issue is because the docker image is not even maintained in docker hub...

@Roiocam
Copy link
Member

Roiocam commented Jun 16, 2024

I emaile IronMQ support earlier. I'll see if they respond.

Thanks.

In addition, i think ironmq team required users to build the docker image from themself, see:

Do you think this information will be helpful?

@pjfanning
Copy link
Contributor Author

The code in https://github.com/iron-io/dockerworker is 8/9 years old. Has checked in jars.
We really have to ask ourselves, who would be using this sort of unmaintained stuff and whether Pekko should support it.

@laglangyue
Copy link
Contributor

I git pull and git tag the image, the manifest will changed to schema-v2.

We have some ways to solve it:

  1. drop this module
  2. git pull, git tag, git push to pekko in dockerhub, This may involve some license issues
  3. Waiting for iron to fix it, but I don't think they will because the Dockerhub image has not been maintained for a long time
    so should we
  4. just disable the test

I aggre to drop the module.

@raboof
Copy link
Member

raboof commented Jun 17, 2024

I agree dropping the module seems reasonable. If someone steps up and offers to revive/maintain the module they can do so outside of the ASF.

Might be worth highlighting on the dev list?

@pjfanning
Copy link
Contributor Author

IronMQ support suggested trying this env variable.
DOCKER_ENABLE_DEPRECATED_PULL_SCHEMA_1_IMAGE=true

This does not help on my Mac laptop. I found the setting in Docker docs but Googling it doesn't return many matches. Even if it worked, it only buys us a bit of time until Docker drops support for the old image format altogether.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants