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

Typo in curl command for a standalone Agent install #204462

Closed
christophercutajar opened this issue Dec 16, 2024 · 1 comment · Fixed by #204660
Closed

Typo in curl command for a standalone Agent install #204462

christophercutajar opened this issue Dec 16, 2024 · 1 comment · Fixed by #204660
Assignees
Labels
bug Fixes for quality problems that affect the customer experience Team:Fleet Team label for Observability Data Collection Fleet team

Comments

@christophercutajar
Copy link

Kibana version: 8.17.0

Elasticsearch version: 8.17.0

Server OS version: N/A

Browser version: N/A

Browser OS version: N/A

Original install method (e.g. download page, yum, from source, etc.): N/A

Describe the bug:

The curl command for downloading the agent has an extra /downloads

Steps to reproduce:

  1. Install an integration
  2. Configure a policy
  3. When adding an agent. go to Run Standalone
  4. Go to Install Elastic Agent on your host

Expected behavior:

curl command should look like:

curl -L -O https://artifacts.elastic.co/downloads/beats/elastic-agent/elastic-agent-8.17.0-linux-x86_64.tar.gz

Screenshots (if relevant):

Image

Errors in browser console (if relevant): N/A

Provide logs and/or server output (if relevant): N/A

Any additional context: N/A

@christophercutajar christophercutajar added the bug Fixes for quality problems that affect the customer experience label Dec 16, 2024
@botelastic botelastic bot added the needs-team Issues missing a team label label Dec 16, 2024
@andrewkroh andrewkroh added the Team:Fleet Team label for Observability Data Collection Fleet team label Dec 16, 2024
@elasticmachine
Copy link
Contributor

Pinging @elastic/fleet (Team:Fleet)

@botelastic botelastic bot removed the needs-team Issues missing a team label label Dec 16, 2024
@Supplementing Supplementing self-assigned this Dec 17, 2024
Supplementing added a commit that referenced this issue Dec 18, 2024
kibanamachine pushed a commit to kibanamachine/kibana that referenced this issue Dec 18, 2024
…astic#204660)

## Summary

Removed the extra `/downloads` portion of the curl command, as it is
already returned in the base url.
Closes elastic#204462

Before:

![image](https://github.com/user-attachments/assets/5a156aca-ea2b-4703-97b3-f7a5e4ae6ab1)

After:

![image](https://github.com/user-attachments/assets/d5c9f0d4-1452-40c1-b8df-4473584ac288)

(cherry picked from commit e29a14d)
kibanamachine added a commit that referenced this issue Dec 18, 2024
…url commands (#204660) (#204779)

# Backport

This will backport the following commits from `main` to `8.x`:
- [[Fleet] removed extraneous '/downloads' in path for curl
commands (#204660)](#204660)

<!--- Backport version: 9.4.3 -->

### Questions ?
Please refer to the [Backport tool
documentation](https://github.com/sqren/backport)

<!--BACKPORT [{"author":{"name":"Mason
Herron","email":"[email protected]"},"sourceCommit":{"committedDate":"2024-12-18T15:37:32Z","message":"[Fleet]
removed extraneous '/downloads' in path for curl commands
(#204660)\n\n## Summary\r\n\r\nRemoved the extra `/downloads` portion of
the curl command, as it is\r\nalready returned in the base
url.\r\nCloses #204462 \r\n\r\nBefore:
\r\n\r\n![image](https://github.com/user-attachments/assets/5a156aca-ea2b-4703-97b3-f7a5e4ae6ab1)\r\n\r\nAfter:
\r\n\r\n![image](https://github.com/user-attachments/assets/d5c9f0d4-1452-40c1-b8df-4473584ac288)","sha":"e29a14d623d1034f6ea12bce679a8fceb95b4cdc","branchLabelMapping":{"^v9.0.0$":"main","^v8.18.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["bug","release_note:skip","Team:Fleet","v9.0.0","backport:prev-minor"],"title":"[Fleet]
removed extraneous '/downloads' in path for curl
commands","number":204660,"url":"https://github.com/elastic/kibana/pull/204660","mergeCommit":{"message":"[Fleet]
removed extraneous '/downloads' in path for curl commands
(#204660)\n\n## Summary\r\n\r\nRemoved the extra `/downloads` portion of
the curl command, as it is\r\nalready returned in the base
url.\r\nCloses #204462 \r\n\r\nBefore:
\r\n\r\n![image](https://github.com/user-attachments/assets/5a156aca-ea2b-4703-97b3-f7a5e4ae6ab1)\r\n\r\nAfter:
\r\n\r\n![image](https://github.com/user-attachments/assets/d5c9f0d4-1452-40c1-b8df-4473584ac288)","sha":"e29a14d623d1034f6ea12bce679a8fceb95b4cdc"}},"sourceBranch":"main","suggestedTargetBranches":[],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","branchLabelMappingKey":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/204660","number":204660,"mergeCommit":{"message":"[Fleet]
removed extraneous '/downloads' in path for curl commands
(#204660)\n\n## Summary\r\n\r\nRemoved the extra `/downloads` portion of
the curl command, as it is\r\nalready returned in the base
url.\r\nCloses #204462 \r\n\r\nBefore:
\r\n\r\n![image](https://github.com/user-attachments/assets/5a156aca-ea2b-4703-97b3-f7a5e4ae6ab1)\r\n\r\nAfter:
\r\n\r\n![image](https://github.com/user-attachments/assets/d5c9f0d4-1452-40c1-b8df-4473584ac288)","sha":"e29a14d623d1034f6ea12bce679a8fceb95b4cdc"}}]}]
BACKPORT-->

Co-authored-by: Mason Herron <[email protected]>
JoseLuisGJ pushed a commit to JoseLuisGJ/kibana that referenced this issue Dec 19, 2024
benakansara pushed a commit to benakansara/kibana that referenced this issue Jan 2, 2025
CAWilson94 pushed a commit to CAWilson94/kibana that referenced this issue Jan 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Fixes for quality problems that affect the customer experience Team:Fleet Team label for Observability Data Collection Fleet team
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants