Skip to content
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

Remove 32 bits artifact for the Elastic Agent #25533

Closed
8 tasks done
ph opened this issue May 4, 2021 · 10 comments
Closed
8 tasks done

Remove 32 bits artifact for the Elastic Agent #25533

ph opened this issue May 4, 2021 · 10 comments
Assignees
Labels
Team:Elastic-Agent Label for the Agent team v7.13.0

Comments

@ph
Copy link
Contributor

ph commented May 4, 2021

We want to make sure that all the platform supported by the Elastic Agent are only 64 bits. Because we want to make sure that all the binaries are compiled with ASLR.

Task:

7.13

7.14

  • Remove 32 bits jobs for the Elastic Agent.
  • Remove 32 bits support in the mage package of the elastic agent.
  • Remove 32 bits artifacts for the fleet server.
@ph ph added v7.13.0 Team:Elastic-Agent Label for the Agent team labels May 4, 2021
@ph ph self-assigned this May 4, 2021
@elasticmachine
Copy link
Collaborator

Pinging @elastic/agent (Team:Agent)

ph added a commit to ph/beats that referenced this issue May 13, 2021
Elastic Agent only support 64 bits system.

Ref: elastic#25533
@dikshachauhan-qasource
Copy link

Hi @ph cc @EricDavisX

Could you please let us know how to validate this ticket.

Thanks
QAS

@EricDavisX
Copy link
Contributor

EricDavisX commented May 20, 2021

@dikshachauhan-qasource you can check the output of the build for the daily snapshots (and BCs and shipped product) and only the x64 variants will exist (if change is working). i'll send you a wiki page we need to update that relates to the artifacts. :)

@ph @mostlyjason did we review with @ferullo to confirm no Endpoint support is required in 7.13 and beyond for x86 (like old Windows 7 that some customers may have?) - just double checking.

Is this intended to impact any Beats builds other than Agent, PH ? Again, just double checking spec for clarity.

Update: I corrected a typo above, I had 7.14 and meant to cite 7.13 as we are modifying the release that is going out now.

@dikshachauhan-qasource
Copy link

Hi @EricDavisX cc @ph

As currently ticket has no merges for 7.13 build so we will wait to validate this ticket till the merges are available for the removal of elastic agent 32 bit artifacts.

Thanks
QAS

@mostlyjason
Copy link

@EricDavisX yes this was requested by Tony to mitigate the ASLR vulnerability

@EricDavisX
Copy link
Contributor

So, there are draft PRs open still but I think other work resolves the desire here. Not sure if we want to track down the teams / prs (and tickets) and do the leg work to close this all out. I don't know any larger coordination needs, like with the Web site or all of the downloadable locations to find artifacts.

@dikshachauhan-qasource
Copy link

Hi @EricDavisX

For now, as per the merges we have validated the 7.14 snapshot artifact link and found 32 bits artifacts links are still available.

Artifact link referred:
https://artifacts-api.elastic.co/v1/search/7.14.0-SNAPSHOT

Screenshot:
image

@ph Could you please confirm us about mage package of the elastic agent. And if we have mistaken anything in validating above merges.

Thanks
QAS

@EricDavisX
Copy link
Contributor

This makes sense then, the above PRs surely are for the Infra job side removal - I just couldn't be sure prior, thanks for noting it Diksha. We can follow up again when the PRs are merged later.

@ph
Copy link
Contributor Author

ph commented Jun 29, 2021

@mgreau I think all the items were achieved on our side for 7.13-14. I will close this one.

@EricDavisX
Copy link
Contributor

Indeed - I don't see us shipping 32-bit arch artifacts as of now. closing it out.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Team:Elastic-Agent Label for the Agent team v7.13.0
Projects
None yet
Development

No branches or pull requests

5 participants