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

[Fleet] Link to download page of current stack version on Agent install instructions #104494

Merged
merged 3 commits into from
Jul 7, 2021

Conversation

Zacqary
Copy link
Contributor

@Zacqary Zacqary commented Jul 6, 2021

Summary

Closes #102491

Links the Go to download page button to the past-releases URL matching the current Kibana version.

Also adds instructions on using the Linux installer over RPM/DEB, as these aren't present on the past release page:
Screen Shot 2021-07-06 at 11 17 58 AM

Checklist

@Zacqary Zacqary added release_note:fix v8.0.0 Feature:Fleet Fleet team's agent central management project Team:Fleet Team label for Observability Data Collection Fleet team v7.14.0 v7.15.0 labels Jul 6, 2021
@Zacqary Zacqary self-assigned this Jul 6, 2021
@Zacqary Zacqary requested a review from a team as a code owner July 6, 2021 16:21
@elasticmachine
Copy link
Contributor

Pinging @elastic/fleet (Team:Fleet)

@elasticmachine
Copy link
Contributor

Pinging @elastic/fleet (Feature:Fleet)

@kibanamachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

Module Count

Fewer modules leads to a faster build time

id before after diff
fleet 483 516 +33

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
fleet 715.8KB 737.3KB +21.5KB

History

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

cc @Zacqary

Copy link
Member

@kpollich kpollich left a comment

Choose a reason for hiding this comment

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

LGTM 🚀

<EuiSpacer size="l" />
<EuiButton
href="https://ela.st/download-elastic-agent"
href={`https://www.elastic.co/downloads/past-releases/elastic-agent-${kibanaVersionURLString}`}
Copy link
Contributor

Choose a reason for hiding this comment

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

one thing that will be annoying, is that this link will be broken for releases which are not published yet (i.e. local development from master/8.0.0, build candidates like the upcoming 7.14.0)

but I'm not sure what we can do about that. @mostlyjason WDYT?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I was concerned about that too, but there doesn't seem to be an easy way to determine in the code whether or not a version is released yet. This might just have to be a thing we deal with in development.

Copy link
Contributor

Choose a reason for hiding this comment

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

sgtm 👍🏻

@Zacqary Zacqary added the auto-backport Deprecated - use backport:version if exact versions are needed label Jul 7, 2021
@mostlyjason
Copy link
Contributor

LGTM. Minor thing: there is only one installer for Linux so it can be singular

import { FLEET_SERVER_PACKAGE } from '../../constants';

import { EnrollmentStepAgentPolicy } from './agent_policy_selection';
import { AdvancedAgentAuthenticationSettings } from './advanced_agent_authentication_settings';

export const DownloadStep = () => {
const kibanaVersion = useKibanaVersion();
const kibanaVersionURLString = useMemo(
Copy link
Contributor

Choose a reason for hiding this comment

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

It's not a blocker for me, but in the main branch this will go to https://www.elastic.co/downloads/past-releases/elastic-agent-8-0-0 which 404's.

I don't know if it's better to leave the existing https://ela.st/download-elastic-agent for snapshot builds or use this invalid URL

One optional enhancement for another time/PR is to use the results from https://artifacts-api.elastic.co/v1/search/8.0-SNAPSHOT/elastic-agent to make a list of links like below (in the flyout?)
Screen Shot 2021-07-07 at 12 10 52 PM

None of these are requirements to ship, IMO; just pointing out what I noticed

@Zacqary Zacqary merged commit e3b6c1f into elastic:master Jul 7, 2021
kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Jul 7, 2021
…ll instructions (elastic#104494)

* [Fleet] Link to download page of current stack version

* Get Kibana version from semver function

* Fix jest
kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Jul 7, 2021
…ll instructions (elastic#104494)

* [Fleet] Link to download page of current stack version

* Get Kibana version from semver function

* Fix jest
@kibanamachine
Copy link
Contributor

💚 Backport successful

Status Branch Result
7.14
7.x

The backport PRs will be merged automatically after passing CI.

kibanamachine added a commit that referenced this pull request Jul 7, 2021
…ll instructions (#104494) (#104730)

* [Fleet] Link to download page of current stack version

* Get Kibana version from semver function

* Fix jest

Co-authored-by: Zacqary Adam Xeper <Zacqary@users.noreply.github.com>
kibanamachine added a commit that referenced this pull request Jul 7, 2021
…ll instructions (#104494) (#104731)

* [Fleet] Link to download page of current stack version

* Get Kibana version from semver function

* Fix jest

Co-authored-by: Zacqary Adam Xeper <Zacqary@users.noreply.github.com>
jloleysens added a commit to jloleysens/kibana that referenced this pull request Jul 8, 2021
…-of-max-results

* 'master' of github.com:elastic/kibana: (36 commits)
  Lower Kibana app bundle limits (elastic#104688)
  [Security Solutions] Fixes bug with the filter query compatibility for transforms (elastic#104559)
  [RAC] Add mapping update logic to RuleDataClient (elastic#102586)
  Fix import workpad (elastic#104722)
  [canvas] Fix Storybook service decorator (elastic#104750)
  [Detection Rules] Add 7.14 rules (elastic#104772)
  [Enterprise Search] Fix beta notification in sidebar (elastic#104763)
  Fix engine routes that are meta engine or non-meta-engine specific (elastic#104757)
  [Fleet] Fix policy revision number getting bumped for no reason (elastic#104696)
  persistable state migrations (elastic#103680)
  [Fleet] Fix add agent in the package policy table (elastic#104749)
  [DOCS] Creates separate doc for security in production (elastic#103973)
  [SO Migration] fix reindex race on multi-instance mode (elastic#104516)
  [Security Solution] Update text in Endpoint Admin pages (elastic#104649)
  [package testing] Decrease timeout to 2 hours (elastic#104668)
  Fix background styling of waterfall chart sidebar tooltip. (elastic#103997)
  [Fleet + Integrations UI] Integrations UI Cleanup (elastic#104641)
  [Fleet] Link to download page of current stack version on Agent install instructions (elastic#104494)
  [Workplace Search] Fix Media Type field preview is unformatted bug (elastic#104684)
  [ML] add marker body (elastic#104672)
  ...

# Conflicts:
#	x-pack/plugins/fleet/public/search_provider.test.ts
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
auto-backport Deprecated - use backport:version if exact versions are needed Feature:Fleet Fleet team's agent central management project release_note:fix Team:Fleet Team label for Observability Data Collection Fleet team v7.14.0 v7.15.0 v8.0.0
Projects
None yet
7 participants