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

[Elastic Agent] Fix sysv init files for deb/rpm installation #22543

Merged
merged 2 commits into from
Nov 16, 2020

Conversation

blakerouse
Copy link
Contributor

@blakerouse blakerouse commented Nov 11, 2020

What does this PR do?

Fixes the sysv init files so it works on deb/rpm systems that use sysv and not systemd.

Why is it important?

So deb/rpm work on sysv init systems.

Checklist

  • My code follows the style guidelines of this project
  • [ ] I have commented my code, particularly in hard-to-understand areas
  • [ ] I have made corresponding changes to the documentation
  • [ ] I have made corresponding change to the default configuration files
  • [ ] I have added tests that prove my fix is effective or that my feature works
  • I have added an entry in CHANGELOG.next.asciidoc or CHANGELOG-developer.next.asciidoc.

Related issues

@blakerouse blakerouse self-assigned this Nov 11, 2020
@botelastic botelastic bot added needs_team Indicates that the issue/PR needs a Team:* label and removed needs_team Indicates that the issue/PR needs a Team:* label labels Nov 11, 2020
@blakerouse blakerouse marked this pull request as ready for review November 11, 2020 14:50
@elasticmachine
Copy link
Collaborator

Pinging @elastic/ingest-management (Team:Ingest Management)

@elasticmachine
Copy link
Collaborator

elasticmachine commented Nov 11, 2020

💚 Build Succeeded

the below badges are clickable and redirect to their specific view in the CI or DOCS
Pipeline View Test View Changes Artifacts preview

Expand to view the summary

Build stats

  • Build Cause: [Pull request #22543 event]

  • Start Time: 2020-11-11T14:51:35.040+0000

  • Duration: 72 min 27 sec

Test stats 🧪

Test Results
Failed 0
Passed 16593
Skipped 1349
Total 17942

@ph ph added the review label Nov 11, 2020
@ph
Copy link
Contributor

ph commented Nov 11, 2020

Can we target 7.x and 7.10 so it goes in 7.10.1?

@ph
Copy link
Contributor

ph commented Nov 11, 2020

@rahulgupta-qasource and @EricDavisX this solve the recent issues we found in debian9 or other based sysv system.

@elasticmachine
Copy link
Collaborator

💚 Flaky test report

Tests succeeded.

Expand to view the summary

Test stats 🧪

Test Results
Failed 0
Passed 16593
Skipped 1349
Total 17942

@blakerouse
Copy link
Contributor Author

/packaging

@blakerouse blakerouse added the needs_backport PR is waiting to be backported to other branches. label Nov 11, 2020
@blakerouse
Copy link
Contributor Author

@ph will backport to 7.10

@EricDavisX
Copy link
Contributor

it would be great to do the possibly minimal work to get the .rpm/.deb tests back running in the e2e-testing repo here elastic/e2e-testing#379
and we could use this chance to add in Debian9 (if the current Debian system under test doesn't have SysV)

Copy link
Contributor

@michalpristas michalpristas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@blakerouse blakerouse merged commit c403d1c into elastic:master Nov 16, 2020
@blakerouse blakerouse deleted the agent-fix-sysv-init branch November 16, 2020 14:46
@blakerouse blakerouse added v7.11.0 and removed needs_backport PR is waiting to be backported to other branches. labels Nov 16, 2020
blakerouse added a commit to blakerouse/beats that referenced this pull request Nov 16, 2020
…#22543)

* Fix sysv init files for deb/rpm.

* Add changelog.

(cherry picked from commit c403d1c)
blakerouse added a commit to blakerouse/beats that referenced this pull request Nov 16, 2020
…#22543)

* Fix sysv init files for deb/rpm.

* Add changelog.

(cherry picked from commit c403d1c)
blakerouse added a commit that referenced this pull request Nov 18, 2020
…b/rpm installation (#22593)

* [Elastic Agent] Fix sysv init files for deb/rpm installation (#22543)

* Fix sysv init files for deb/rpm.

* Add changelog.

(cherry picked from commit c403d1c)

* Empty commit, run tests.
blakerouse added a commit that referenced this pull request Nov 18, 2020
…#22594)

* Fix sysv init files for deb/rpm.

* Add changelog.

(cherry picked from commit c403d1c)
@ghost
Copy link

ghost commented Dec 2, 2020

Hi @EricDavisX /@ph

We are blocked to validate this ticket for 7.10.1 BC1 .rpm(on CentOS7 machine) and .deb(on Debian9 machine) packages on 7.10.1 BC1 Kibana cloud environment(deployed from staging platform) due to bug #22744

Build details:
Kibana Commit: 1d4cfde288ede0c4bc6277b4d589ca0cb96b231f
Build : 36050
7.10.1 BC1 Linux .rpm agent package link: https://staging.elastic.co/7.10.1-ce3b40e6/downloads/beats/elastic-agent/elastic-agent-7.10.1-x86_64.rpm
7.10.1 BC1 Linux .deb agent package link: https://staging.elastic.co/7.10.1-ce3b40e6/downloads/beats/elastic-agent/elastic-agent-7.10.1-amd64.deb

Once the above bug #22744 is fixed, we will validate this ticket.

@EricDavisX
Copy link
Contributor

discussed with team we will target additional fixes in 7.11.0 and 7.10.2 - we are past the date for 7.10.1 fixes to routinely be discussed and since it is not functioning in 7.10 we aren't any worse than we were. we can do a thorough testing run in 22744 in later cycle.

@ghost
Copy link

ghost commented Jan 12, 2021

Hi @EricDavisX

We have performed regression testing for Linux deb/rpm on 7.10.2 BC1 Kibana cloud build. Build details is as follows:

Platform: Staging
Version: 7.10.2
Commit: a0b7936
Build number: 36136
Artifact: https://staging.elastic.co/7.10.2-e6b2b8c2/summary-7.10.2.html

We have execute test cases under test plan execution :

Please let us know if anything else is required to be covered.

Thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
5 participants