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

Manual backport of #16291 #16410

Merged
merged 1 commit into from
Feb 23, 2023
Merged

Manual backport of #16291 #16410

merged 1 commit into from
Feb 23, 2023

Conversation

nathancoleman
Copy link
Member

@nathancoleman nathancoleman commented Feb 23, 2023

Description

This fixes an error causing Vercel to fail on release/1.15.x branches.
The error was introduced when the backport of #16291, #16398, failed to remove a file that was removed in the initial PR.

CleanShot 2023-02-23 at 16 32 44@2x

Testing & Reproduction steps

Links

Example Vercel failure

PR Checklist

  • updated test coverage
  • external facing docs updated
  • not a security concern

@github-actions github-actions bot added the type/docs Documentation needs to be created/updated/clarified label Feb 23, 2023
@nathancoleman nathancoleman added the backport/1.15 This release series is no longer active on CE. Use backport/ent/1.15. label Feb 23, 2023
* initial commit

* initial commit

* Overview updates

* Overview page improvements

* More Overview improvements

* improvements

* Small fixes/updates

* Updates

* Overview updates

* Nav data

* More nav updates

* Fix

* updates

* Updates + tip test

* Directory test

* refining

* Create restructure w/ k8s

* Single usage page

* Technical Specification

* k8s pages

* typo

* L7 traffic management

* Manage connections

* k8s page fix

* Create page tab corrections

* link to k8s

* intentions

* corrections

* Add-on intention descriptions

* adjustments

* Missing </CodeTabs>

* Diagram improvements

* Final diagram update

* Apply suggestions from code review

Co-authored-by: trujillo-adam <47586768+trujillo-adam@users.noreply.github.com>
Co-authored-by: Tu Nguyen <im2nguyen@users.noreply.github.com>
Co-authored-by: David Yu <dyu@hashicorp.com>

* diagram name fix

* Fixes

* Updates to index.mdx

* Tech specs page corrections

* Tech specs page rename

* update link to tech specs

* K8s - new pages + tech specs

* k8s - manage peering connections

* k8s L7 traffic management

* Separated establish connection pages

* Directory fixes

* Usage clean up

* k8s docs edits

* Updated nav data

* CodeBlock Component fix

* filename

* CodeBlockConfig removal

* Redirects

* Update k8s filenames

* Reshuffle k8s tech specs for clarity, fmt yaml files

* Update general cluster peering docs, reorder CLI > API > UI, cross link to kubernetes

* Fix config rendering in k8s usage docs, cross link to general usage from k8s docs

* fix legacy link

* update k8s docs

* fix nested list rendering

* redirect fix

* page error

---------

Co-authored-by: trujillo-adam <47586768+trujillo-adam@users.noreply.github.com>
Co-authored-by: Tu Nguyen <im2nguyen@users.noreply.github.com>
Co-authored-by: David Yu <dyu@hashicorp.com>
Co-authored-by: Tu Nguyen <im2nguyen@gmail.com>
@nathancoleman nathancoleman requested a review from a team February 23, 2023 21:48
@nathancoleman nathancoleman requested review from a team as code owners February 23, 2023 21:48
@nathancoleman nathancoleman requested review from sarahethompson and claire-labry and removed request for a team February 23, 2023 21:48
@nathancoleman nathancoleman changed the title Add cluster peering configuration page to nav Manual backport of #16291 Feb 23, 2023
@nathancoleman nathancoleman changed the base branch from main to release/1.15.x February 23, 2023 21:50
@nathancoleman nathancoleman removed the backport/1.15 This release series is no longer active on CE. Use backport/ent/1.15. label Feb 23, 2023
Copy link
Contributor

@boruszak boruszak left a comment

Choose a reason for hiding this comment

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

Thank you so much for you help with this issue, Nathan!

Approving to allow the merge.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type/docs Documentation needs to be created/updated/clarified
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants