Skip to content

Add WSO2 API Manager Analytics Worker Docker images to Official Docker Library#6044

Closed
chirangaalwis wants to merge 3 commits intodocker-library:masterfrom
chirangaalwis:docker-official-images-ldf-wso2apim-analytics-worker
Closed

Add WSO2 API Manager Analytics Worker Docker images to Official Docker Library#6044
chirangaalwis wants to merge 3 commits intodocker-library:masterfrom
chirangaalwis:docker-official-images-ldf-wso2apim-analytics-worker

Conversation

@chirangaalwis
Copy link
Copy Markdown

@chirangaalwis chirangaalwis commented Jun 7, 2019

Checklist for Review

  • associated with or contacted upstream?
    • Yes, this request has been made by WSO2 Inc. I am one of the maintainers of WSO2 product Docker images.
  • does it fit into one of the common categories? ("service", "language stack", "base distribution")
    • Service
  • is it reasonably popular, or does it solve a particular use case well?
    • WSO2 was named a Leader in The Forrester Wave™: API Management Solutions, Q4 2018 Report.
  • does a documentation PR exist? (should be reviewed and merged at roughly the same time so that we don't have an empty image page on the Hub for very long)
  • dockerization review for best practices and cache gotchas/improvements (ala the official review guidelines)?
  • 2+ dockerization review?
  • existing official images have been considered as a base? (ie, if foobar needs Node.js, has FROM node:... instead of grabbing node via other means been considered?)
    • Yes, we have used Ubuntu 18.04 LTS based Official Docker image with AdoptOpenJDK 8 HotSpot JVM pre-installed ( adoptopenjdk:8u212-b03-jdk-hotspot).
  • if FROM scratch, tarballs only exist in a single commit within the associated history?
  • passes current tests? any simple new tests that might be appropriate to add? (https://github.com/docker-library/official-images/tree/master/test)

@chirangaalwis
Copy link
Copy Markdown
Author

Closing this PR and integrating the WSO2 API Manager Analytics Worker profile Docker images under the wso2am image repository (please see #6043 (comment)).

@chirangaalwis
Copy link
Copy Markdown
Author

Re-opening this PR based on the comment #6043 (comment).

@chirangaalwis chirangaalwis reopened this Jun 11, 2019
@chirangaalwis
Copy link
Copy Markdown
Author

As per comment #6043 (comment), I am closing this PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants