docs: new k8 docs + new Operator doc - #420
Conversation
Sorry, something went wrong.
Deploying localstack-docs with
|
| Latest commit: | 9e5d1dc |
| Status: | ✅ Deploy successful! |
| Preview URL: | https://874e9621.localstack-docs.pages.dev |
| Branch Preview URL: | https://docs-new-k8-section.localstack-docs.pages.dev |
Sorry, something went wrong.
fyi @simonrw @mmaureenliu keep in mind the current manual markup table we added is not responsive (normal behavior for table markup): https://6852d11b.localstack-docs.pages.dev/aws/enterprise/kubernetes/kubernetes-operator/#permissions We need to incorporate the same table component we're using in aws API Coverage tables, such as this one: cc @HarshCasper |
Sorry, something went wrong.
simonrw
left a comment
There was a problem hiding this comment.
In general a great restructuring of the Kubernetes section, thank you! However I have a few comments
Sorry, something went wrong.
mmaureenliu
left a comment
There was a problem hiding this comment.
I really LOVE this new doc!
Made some comments and suggestions and pointed a few questions to @simonrw . All in all this is great work. Thanks everybody involved! Look forward to seeing the final publication. We'll make further improvements based on user feedback but this is an amazing start.
Sorry, something went wrong.
Co-authored-by: Simon Walker <simon.walker@localstack.cloud>
Co-authored-by: Simon Walker <simon.walker@localstack.cloud>
Updated installation instructions for LocalStack Pro and added note about customizing the chart with a values.yaml file.
Added a section on certificate issues when spawning child pods, including error details and a solution for configuring NO_PROXY.
Co-authored-by: Simon Walker <simon.walker@localstack.cloud>
quetzalliwrites
left a comment
There was a problem hiding this comment.
Ok, ready for your review again @mmaureenliu @simonrw @nik-localstack @localstack/unicorn
Sorry, something went wrong.
mmaureenliu
left a comment
There was a problem hiding this comment.
I'm happy for it to be merges. I've left a couple comments for information only and a couple recommendations, but nothing show stopper.
Thanks everybody for this great job!
Sorry, something went wrong.
|
Hola @simonrw, can you please approve this PR too? Since you requested changes and are a code service owner, we can't merge until you approve :) |
Sorry, something went wrong.
simonrw
left a comment
There was a problem hiding this comment.
A few previous review comments that got missed in the cleanup I think, but once these are resolved I'm really happy with these docs. Thanks for pushing this through!
Sorry, something went wrong.
Co-authored-by: Simon Walker <simon.walker@localstack.cloud>
Co-authored-by: Simon Walker <simon.walker@localstack.cloud>
Co-authored-by: Simon Walker <simon.walker@localstack.cloud>
Preview URL w/ new k8 docs:
https://34fdda47.localstack-docs.pages.dev/aws/enterprise/kubernetes
Resolves: