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

Append serverless project ID to Support URL #171448

Merged

Conversation

tsullivan
Copy link
Member

@tsullivan tsullivan commented Nov 16, 2023

This PR updates the URL to the Elastic Support Portal registered by the Cloud plugin, to include the configured deployment ID as a querystring parameter.

  1. On serverless deployments, we set the projects unique identifier with ?serverless_project_id=123ABC
  2. On stateful cloud deployments, we set the deployment's unique identifier with ?cloud_deployment_id=123ABC
  3. On on-prem deployments functionality shall remain unchanged.

Where this link can be found in the UI:
image

@tsullivan tsullivan force-pushed the serverless/support-link-add-deployment-id branch from 42369ab to 199c86b Compare November 16, 2023 22:07
@tsullivan tsullivan self-assigned this Nov 16, 2023
@tsullivan tsullivan requested a review from a team November 16, 2023 22:29
@tsullivan tsullivan added Team:SharedUX Team label for AppEx-SharedUX (formerly Global Experience) Project:Serverless Work as part of the Serverless project for its initial release release_note:skip Skip the PR/issue when compiling release notes labels Nov 16, 2023
@tsullivan tsullivan marked this pull request as ready for review November 16, 2023 22:31
@tsullivan tsullivan requested a review from a team as a code owner November 16, 2023 22:31
@elasticmachine
Copy link
Contributor

Pinging @elastic/appex-sharedux (Team:SharedUX)

Copy link
Contributor

@pgayvallet pgayvallet left a comment

Choose a reason for hiding this comment

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

LGTM

Comment on lines 106 to 113
let supportUrl = ELASTIC_SUPPORT_LINK;
if (this.config.serverless?.project_id) {
// serverless projects use config.id and config.serverless.project_id
supportUrl += '?serverless_project_id=' + this.config.serverless.project_id;
} else if (this.config.id) {
// non-serverless Cloud projects only use config.id
supportUrl += '?cloud_deployment_id=' + this.config.id;
}
Copy link
Contributor

Choose a reason for hiding this comment

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

NIT: I would extract that to an getSupportUrl(config: CloudConfigType): string helper function

Copy link
Contributor

@vadimkibana vadimkibana left a comment

Choose a reason for hiding this comment

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

Code LGTM

@jloleysens
Copy link
Contributor

💯
Screenshot 2023-11-21 at 13 41 20

@elastic elastic deleted a comment from kibana-ci Nov 21, 2023
@elastic elastic deleted a comment from kibana-ci Nov 21, 2023
@tsullivan tsullivan enabled auto-merge (squash) November 21, 2023 21:39
@kibana-ci
Copy link
Collaborator

💛 Build succeeded, but was flaky

Failed CI Steps

Metrics [docs]

Module Count

Fewer modules leads to a faster build time

id before after diff
cloud 10 11 +1

Page load bundle

Size of the bundles that are downloaded on every page load. Target size is below 100kb

id before after diff
cloud 6.0KB 6.2KB +190.0B

History

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

cc @tsullivan

@tsullivan tsullivan merged commit 641983f into elastic:main Nov 22, 2023
28 checks passed
@kibanamachine kibanamachine added v8.12.0 backport:skip This commit does not require backporting labels Nov 22, 2023
@tsullivan tsullivan deleted the serverless/support-link-add-deployment-id branch April 30, 2024 21:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport:skip This commit does not require backporting Project:Serverless Work as part of the Serverless project for its initial release release_note:skip Skip the PR/issue when compiling release notes Team:SharedUX Team label for AppEx-SharedUX (formerly Global Experience) v8.12.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants