-
Notifications
You must be signed in to change notification settings - Fork 16
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
ToDo #3
Comments
Are we okay with just |
I suspect it's not a big deal at all, as it's controlled by a single switch. If it turns out to be a problem, We'll restrict to x64 only. |
It may not be a big deal to flip the switch, but we still have to test, support and maintain x86. Please only create for x86 if there is a strong need. I do not think this has been established. |
Hi @sophiec20, I see. Beats team publishes x86 artifacts, so it was reasonable to support them. I'll mention this in our sync-up with them today. In 85c4208, I just pushed, I defaulted bitness to UPD: I just spoke with @urso. He mentioned that usage of x86 binaries is quite high, so for now they stay as supported variant. x64 will be default for CLI though. |
Everything is accounted for in tickets and comments |
TODOs
/q
,/qn
,/q+
)./a
).all
build target - builds all known installers. 4cbc102Grab short description of a beat from itsDependency removed in 92fa202.README.md
file.Cache used ProductId (guid) -> version map inDeferred.config.yaml
.elastic/elasticsearch-net-abstractions
to talk to artifacts-api. Elasticsearch Windows MSI package #59Add support forNo actionlatest
, signifying latest version of the artifact (container).Might not be trivial: https://github.com/elastic/infra/issues/14645, https://github.com/elastic/infra/issues/5193, Elasticsearch latest not available elasticsearch-docker#215 (comment)
Converted to tickets:
Notify Beats team that installer build process depends on beat'sREADME.md
file contents, specifically line 3, containing beat description. Discuss alternatives to make it less brittle.README.md
dependency removed in 92fa202, others remain, see Installer dependencies on metadata in Beats repository #4.No
). Migrate configuration on upgrade #13SimpleExec
with runner capable of returning exit codes andstdout
/stderr
(possiblynullean/proc
). Process runner replacement #16References
The text was updated successfully, but these errors were encountered: