Skip to content
This repository was archived by the owner on Sep 17, 2024. It is now read-only.

Install jq tool in elastic-agent container to parse ES output#2141

Merged
mdelapenya merged 1 commit intoelastic:mainfrom
ChrsMark:fix_jq_issue
Feb 11, 2022
Merged

Install jq tool in elastic-agent container to parse ES output#2141
mdelapenya merged 1 commit intoelastic:mainfrom
ChrsMark:fix_jq_issue

Conversation

@ChrsMark
Copy link
Copy Markdown
Member

@ChrsMark ChrsMark commented Feb 11, 2022 •
edited by mdelapenya
Loading

What does this PR do?

Tests for elastic-agent were failing because of:

(23) Failed writing body
/etc/writer.sh: 12: jq: not found
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100 13884  100 13884    0     0  1129k      0 --:--:-- --:--:-- --:--:-- 1129k
curl: (23) Failed writing body (0 != 13884)

Closes #1992

@ChrsMark
Install jq tool in elastic-agent container to parse ES output
3749d9f 
Signed-off-by: ChrsMark <chrismarkou92@gmail.com>
@ChrsMark ChrsMark requested a review from mdelapenya February 11, 2022 10:57
@ChrsMark ChrsMark self-assigned this Feb 11, 2022
@mergify
Copy link
Copy Markdown
Contributor

mergify bot commented Feb 11, 2022

This pull request does not have a backport label. Could you fix it @ChrsMark? 🙏
To fixup this pull request, you need to add the backport labels for the needed
branches, such as:

  • backport-v./d./d./d is the label to automatically backport to the 7./d branch. /d is the digit
    NOTE: backport-skip has been added to this pull request.

@mergify mergify bot added the backport-skip Skip notification from the automated backport with mergify label Feb 11, 2022
@ChrsMark ChrsMark mentioned this pull request Feb 11, 2022
Closed
@elasticmachine
Copy link
Copy Markdown
Contributor

elasticmachine commented Feb 11, 2022

💚 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 preview

Expand to view the summary

Build stats

  • Start Time: 2022-02-11T10:57:56.352+0000

  • Duration: 49 min 11 sec

Test stats 🧪

Test Results
Failed 0
Passed 253
Skipped 0
Total 253

💚 Flaky test report

Tests succeeded.

🤖 GitHub comments

To re-run your PR in the CI, just comment with:

  • /test : Re-trigger the build.

@mdelapenya mdelapenya added backport-v7.16.0 Automated backport with mergify backport-v7.17.0 Automated backport with mergify backport-v8.0.0 Automated backport with mergify and removed backport-skip Skip notification from the automated backport with mergify labels Feb 11, 2022
@mdelapenya mdelapenya merged commit 05bf7b1 into elastic:main Feb 11, 2022
mergify bot pushed a commit that referenced this pull request Feb 11, 2022
@ChrsMark
Install jq tool in elastic-agent container to parse ES output (#2141)
47500dd 
Signed-off-by: ChrsMark <chrismarkou92@gmail.com>
(cherry picked from commit 05bf7b1)
@mergify mergify bot mentioned this pull request Feb 11, 2022
Merged
mergify bot pushed a commit that referenced this pull request Feb 11, 2022
@ChrsMark
Install jq tool in elastic-agent container to parse ES output (#2141)
1d38ad9 
Signed-off-by: ChrsMark <chrismarkou92@gmail.com>
(cherry picked from commit 05bf7b1)
@mergify mergify bot mentioned this pull request Feb 11, 2022
Merged
mergify bot pushed a commit that referenced this pull request Feb 11, 2022
@ChrsMark
Install jq tool in elastic-agent container to parse ES output (#2141)
c8ffc42 
Signed-off-by: ChrsMark <chrismarkou92@gmail.com>
(cherry picked from commit 05bf7b1)
@mergify mergify bot mentioned this pull request Feb 11, 2022
Merged
@mdelapenya
Copy link
Copy Markdown
Contributor

mdelapenya commented Feb 11, 2022

Thank you @ChrsMark for your time debugging this issue! Merged and backported to the maintenance branches

mdelapenya pushed a commit that referenced this pull request Feb 11, 2022
@mergify @ChrsMark
Install jq tool in elastic-agent container to parse ES output (#2141) (…
f473b12 
…#2144)

Signed-off-by: ChrsMark <chrismarkou92@gmail.com>
(cherry picked from commit 05bf7b1)

Co-authored-by: Chris Mark <chrismarkou92@gmail.com>
mdelapenya pushed a commit that referenced this pull request Feb 14, 2022
@mergify @ChrsMark
Install jq tool in elastic-agent container to parse ES output (#2141) (…
969344a 
…#2143)

Signed-off-by: ChrsMark <chrismarkou92@gmail.com>
(cherry picked from commit 05bf7b1)

Co-authored-by: Chris Mark <chrismarkou92@gmail.com>
mdelapenya pushed a commit that referenced this pull request Feb 15, 2022
@mergify @ChrsMark
Install jq tool in elastic-agent container to parse ES output (#2141) (…
5cba3c7 
…#2142)

Signed-off-by: ChrsMark <chrismarkou92@gmail.com>
(cherry picked from commit 05bf7b1)

Co-authored-by: Chris Mark <chrismarkou92@gmail.com>
@ChrsMark ChrsMark mentioned this pull request Mar 15, 2022
Merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

1 more reviewer

@mdelapenya mdelapenya mdelapenya approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@ChrsMark ChrsMark

Labels

backport-v7.16.0 Automated backport with mergify backport-v7.17.0 Automated backport with mergify backport-v8.0.0 Automated backport with mergify

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

k8s-autodiscover for elastic-agent fails

3 participants

@ChrsMark @elasticmachine @mdelapenya