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
CONSOLE-2381: Support dynamic demo plugin deployment on cluster #7471
CONSOLE-2381: Support dynamic demo plugin deployment on cluster #7471
Conversation
/hold |
b9fd987
to
8ebecc9
Compare
a2d286e
to
3dc8bcb
Compare
/hold cancel |
/retest |
@jhadvig Thanks for the review! PR updated, please have a second look. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@vojtechszocs lets suqsh this into a single commit and we can tag it 👍
/approve
51073ad
to
f1fb348
Compare
/lgtm |
/approve |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: jhadvig, rhamilto, vojtechszocs The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
/retest Please review the full test history for this PR and help us cut down flakes. |
3 similar comments
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
The image is based on official Node.js image, using
http-server.sh
script as the entry point and running asnode
user.@spadgett Built image size is
985MB
, is this OK or should we consider usingnode:alpine
as the base image to reduce its size?When building the Docker image, we need the whole Console repo as the build context. This is because
frontend/codegen.yml
is referencing GraphQL schemas atpkg/graphql
.The plugin HTTP server runs at port 9001 with caching disabled and CORS enabled.