Skip to content

Revert "ci: update redirects with replace directive in s3 bucket" - #15118

Merged
thaJeztah merged 1 commit into
docker:masterfrom
crazy-max:revert-redirects
Jul 13, 2022
Merged

Revert "ci: update redirects with replace directive in s3 bucket"#15118
thaJeztah merged 1 commit into
docker:masterfrom
crazy-max:revert-redirects

Conversation

@crazy-max

@crazy-maxcrazy-max commented Jul 13, 2022

Copy link
Copy Markdown
Member

This reverts commit ba3ca84.

Signed-off-by: CrazyMax crazy-max@users.noreply.github.com

This reverts commit ba3ca84.
Signed-off-by: CrazyMax <crazy-max@users.noreply.github.com>
@crazy-maxcrazy-max changed the title Revert "Merge pull request #15109 from crazy-max/redirects"Revert "ci: update redirects with replace directive in s3 bucket"Jul 13, 2022
@thaJeztah

Copy link
Copy Markdown
Member

For those curious; we tried deploying this, but looks like

  • the CDN didn't like it
  • uploading the redirects takes a LOOOOONG time (30 minutes), which is too much

@netlify

netlifyBot commented Jul 13, 2022

Copy link
Copy Markdown

Deploy Preview for docsdocker ready!

Built without sensitive environment variables

NameLink
🔨 Latest commit387a050
🔍 Latest deploy loghttps://app.netlify.com/sites/docsdocker/deploys/62cef6f803ab400008e0f525
😎 Deploy Previewhttps://deploy-preview-15118--docsdocker.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@thaJeztahthaJeztah left a comment

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.

LGTM 😢

@netlify

netlifyBot commented Jul 13, 2022

Copy link
Copy Markdown

Deploy Preview for docsdocker ready!

Built without sensitive environment variables

NameLink
🔨 Latest commit4ba8f8d
🔍 Latest deploy loghttps://app.netlify.com/sites/docsdocker/deploys/62cef719c2e7ea0008661728
😎 Deploy Previewhttps://deploy-preview-15118--docsdocker.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@thaJeztah
thaJeztah merged commit c065bad into docker:masterJul 13, 2022
@crazy-max
crazy-max deleted the revert-redirects branch July 18, 2022 14:07
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@crazy-max@thaJeztah