-
Notifications
You must be signed in to change notification settings - Fork 31
Update workflow to build artifact for ARM64 #228
Update workflow to build artifact for ARM64 #228
Conversation
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
@@ -32,9 +32,9 @@ jobs: | |||
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.
This step is missing in the release flow
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.
Added, although I think we only need to test in CI, shouldn't need to test in CD?
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.
it may be a requirement from infra team, guess all the plugins are doing this way
Do we have plan to add chromium artifacts for macOS? |
I don't think so, ODFE doesn't have a mac version |
developers may want it |
…arch#228) * Add arm chrome to workflows * Update release workflow * Fix workflow * Add puppeteer-core as it's not included in kibana-oss * Address comments
…arch#228) * Add arm chrome to workflows * Update release workflow * Fix workflow * Add puppeteer-core as it's not included in kibana-oss * Address comments
Issue #, if available:
Description of changes:
puppeteer-core
as it's not included in kibana-ossArtifact names:
opendistroReportsKibana-1.12.0.0-linux-x64.zip
opendistroReportsKibana-1.12.0.0-linux-arm64.zip
opendistroReportsKibana-1.12.0.0-windows-x64.zip
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.