Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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
chore: secure hermetic-build docker image #3196
base: main
Are you sure you want to change the base?
chore: secure hermetic-build docker image #3196
Changes from 1 commit
5f5e0bd
15a19c5
ff0fc1a
9a25dda
66d7f45
d12877c
4885459
b97c209
3f94f0b
08fe2cd
539922a
1a34741
43f1ac0
1dc3629
90dafe3
3f3deec
b730a4b
db2e8e7
fb98222
32fffb7
51544a3
e9a5df4
8f0ac9b
2c35db2
d09124f
34835a5
a3490e2
efeff60
ae0f349
9177111
8797e30
aa0fe85
bce332d
d0a6da1
11a81d3
565afda
dd275ac
2e5067d
32ac053
054bf42
87a2e09
42f331e
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
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.
In order to create a SEA, do we have to bundle source code into a single js file?
Can we combine the two steps into one?
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.
Yes, bundling is necessary:
sdk-platform-java/.cloudbuild/library_generation/library_generation.Dockerfile
Lines 39 to 40 in fb98222
From the docs: The single executable application feature currently only supports running a single embedded script using the CommonJS module system.
Do you mean using a single
RUN
+&&
?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.
I was referring whether we can build a SEA from multiple js source code, looks like it has to be bundled together first.
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.
Which version of the gcloud?
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.
Good point, I'll figure this out in the follow up (enhancements) since it is now out of scope.