Skip to content

Move over to Google Storage vs AWS#44685

Merged
joelgriffith merged 3 commits intoelastic:masterfrom
joelgriffith:chore/move-chromium-builds-gcp
Sep 6, 2019
Merged

Move over to Google Storage vs AWS#44685
joelgriffith merged 3 commits intoelastic:masterfrom
joelgriffith:chore/move-chromium-builds-gcp

Conversation

@joelgriffith
Copy link
Copy Markdown
Contributor

Summary

Move our chromium builds to be served on Google Storage. I can see this potentially being a breaking change if users had previously whitelisted our old storage URL, so I might drop the 7.5.0 target if so otherwise I'd like to migrate over and start removing our older builds.

@joelgriffith joelgriffith added zDeprecated Feature:Reporting Use Reporting:Screenshot, Reporting:CSV, or Reporting:Framework instead v8.0.0 release_note:skip Skip the PR/issue when compiling release notes Team:Stack Services v7.5.0 labels Sep 3, 2019
@elasticmachine
Copy link
Copy Markdown
Contributor

Pinging @elastic/kibana-stack-services

@elasticmachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

export const paths = {
archivesPath: path.resolve(__dirname, '../../../.chromium'),
baseUrl: 'https://s3.amazonaws.com/headless-shell/',
baseUrl: 'https://storage.googleapis.com/headless_shell/',
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

would it be overkill to make this configurable?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I'm leaning towards yes: the checksums make it impossible to implement this yourself (in the case of our users), and having different vendors for distributing this means having to maintain multiple places + another layer of debugging for services in case someone does something goofy. Honestly I don't envision this changing much so I'm ok with making it an application artifact versus a configuration artifact.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks like we understand now that the baseUrl is only a factor in a dev and build environment :D

@tsullivan
Copy link
Copy Markdown
Member

I think I lack the context for this. Is there an issue we can link in the description?

@elasticmachine
Copy link
Copy Markdown
Contributor

💔 Build Failed

Copy link
Copy Markdown
Member

@tsullivan tsullivan left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Copy Markdown
Contributor

@spalger spalger left a comment

Choose a reason for hiding this comment

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

LGTM

@elasticmachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

@joelgriffith joelgriffith merged commit f8ba023 into elastic:master Sep 6, 2019
tylersmalley pushed a commit to tylersmalley/kibana that referenced this pull request Jul 13, 2020
tylersmalley added a commit that referenced this pull request Jul 13, 2020
Co-authored-by: Joel Griffith <joel.griffith@elastic.co>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release_note:skip Skip the PR/issue when compiling release notes v7.9.0 v8.0.0 zDeprecated Feature:Reporting Use Reporting:Screenshot, Reporting:CSV, or Reporting:Framework instead

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants