-
Notifications
You must be signed in to change notification settings - Fork 456
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
Example containers using old images #417
Comments
@navendu-pottekkat can you have a look at this please? I'm also curious whether we have a set of updated images for the same which will be potentially used as a replacement? |
@Neilblaze Thank you for the ping. @allentc Thank you for opening this issue. @Neilblaze yes, these could be updated. I think we can use the latest LTS versions of each of the images. Prior to making a pull request, please test and check if everything is working as intended. We can use the existing images but bump the version. I don't there would be anything else to change but look out for compatibility issues. |
Prometheus & Grafana are for the |
I don't think all-in-one has any effect, we can consider deleting it |
I'm a bit confused about whether to remove Prometheus and other images entirely from the compose file. Would you mind sharing some light on this? And else, if I want to upgrade them in general, where can I get the latest images that would be used as potential replacement? cc: @juzhiyuan , @soulbird |
To confirm: "them" - do you mean Prometheus, APISIX, and other components? For this issue, welcome to file a PR to upgrade them. |
@Neilblaze You can use the later versions of the same images from Docker Hub. |
Hey @pottekkat can you assign this issue to me and let me upgrade examples to the latest lts |
Sure @Bloodraven21 . Assigned. |
i have created a pr |
Some of the images that appear in example/docker-compose.yml are very old. Example:
I'd offer a pull request to update images but I haven't found any compatibility matrix or other information on what has been qualified for use with APISIX.
The text was updated successfully, but these errors were encountered: