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

Go workspaces blog post (take 2) #45626

Closed
wants to merge 1 commit into from
Closed

Conversation

thockin
Copy link
Member

@thockin thockin commented Mar 21, 2024

A blog post on Go workspaces in k/k

Replay of #45358, revert #45625 - apparently we jumped the gun and had to revert it. This restores it.

/cc @katcosgrove @natalisucks @sftim

@k8s-ci-robot k8s-ci-robot added cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. size/L Denotes a PR that changes 100-499 lines, ignoring generated files. area/blog Issues or PRs related to the Kubernetes Blog subproject language/en Issues or PRs related to English language sig/docs Categorizes an issue or PR as relevant to SIG Docs. labels Mar 21, 2024
@sftim
Copy link
Contributor

sftim commented Mar 21, 2024

@thockin - can you explain in the PR description why we're making this change?

/lgtm
/approve

/hold

PR description isn't clear; ideally, clarify and then unhold

@k8s-ci-robot k8s-ci-robot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Mar 21, 2024
Copy link

netlify bot commented Mar 21, 2024

Pull request preview available for checking

Name Link
🔨 Latest commit 2fc9937
🔍 Latest deploy log https://app.netlify.com/sites/kubernetes-io-main-staging/deploys/6601f5f28d858000088b595c
😎 Deploy Preview https://deploy-preview-45626--kubernetes-io-main-staging.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 configuration.

@k8s-ci-robot k8s-ci-robot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Mar 21, 2024
@k8s-ci-robot
Copy link
Contributor

LGTM label has been added.

Git tree hash: d73e6e0dcaa772e4845018486ad7fbe95e5c6810

@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: sftim

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@k8s-ci-robot k8s-ci-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Mar 21, 2024
@thockin
Copy link
Member Author

thockin commented Mar 21, 2024

Do I need to fix the date?

@sftim
Copy link
Contributor

sftim commented Mar 21, 2024

I don't have context about the article yank; to me, it was fine to publish as-was.

@natalisucks
Copy link
Contributor

natalisucks commented Mar 23, 2024

@thockin thanks so much for resubmitting this blog! It was also great to meet you at KubeCon 🚀

Do I need to fix the date?

Can you please set the publication date for April 19, 2024? 2024-04-19

---

**Author:** Tim Hockin (Google)

Copy link
Contributor

Choose a reason for hiding this comment

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

We could add some text to explain that this is reposting the content published in March at https://www.k8s.dev/blog/2024/03/19/go-workspaces-in-kubernetes/

I don't understand our overall intent here; the improvement is great, but it's already delivered and end users won't notice it. We can have it as a post-release blog article but I don't understand why we would.

Copy link
Contributor

Choose a reason for hiding this comment

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

This is to be published post-release because it was handled and tracked by the Release Comms team as a feature blog. It should not have been published without their knowledge or without them assigning a publication date. The original PR included all of that context. We reverted because it was, to the Release Team, an unauthorized early publication.

If it should never have been a feature blog and thus should never have been under the control of Release Comms, that's fine and it can be published whenever you and @thockin please, but as long as Release Comms owns it, Release Comms needs to be kept in the loop and approve of publication.

No objection to removing this from Release Comms ownership if it's regarding a feature that is already live and they don't have a problem with it, but everyone has to be informed of that.

Copy link
Member Author

Choose a reason for hiding this comment

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

I truly truly do not care and am happy to roll with . It's not a "feature" of 1.30 - it's in the codebase now and any developer using k/k is subject to it. That said, it was a KEP and tracked.

Copy link
Contributor

Choose a reason for hiding this comment

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

We (The 1.30 Comms team) tracked it in the 1.30 project board and scheduled it as a feature blog due to it being a KEP. As previously stated, we just need someone to inform us whether or not this is remaining with the Comms Team as a feature blog.

Copy link
Member Author

Choose a reason for hiding this comment

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

For the less clueful - it's already published at https://www.k8s.dev/blog/2024/03/19/go-workspaces-in-kubernetes/ - so if we say this is NOT a feature blog, do I just close this PR and be done? Any other implications?

Copy link
Contributor

Choose a reason for hiding this comment

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

For the less clueful - it's already published at https://www.k8s.dev/blog/2024/03/19/go-workspaces-in-kubernetes/ - so if we say this is NOT a feature blog, do I just close this PR and be done? Any other implications?

I think we just close it and leave the contributor site article published. I don't think it's a post-release blog because the improvement isn't awaiting the upcoming v1.30 release.

Any objections to that?

Copy link
Contributor

Choose a reason for hiding this comment

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

We have no objections, go ahead and close it.

@k8s-ci-robot k8s-ci-robot removed the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Mar 25, 2024
@k8s-ci-robot k8s-ci-robot requested a review from sftim March 25, 2024 16:20
@k8s-ci-robot
Copy link
Contributor

New changes are detected. LGTM label has been removed.

@thockin
Copy link
Member Author

thockin commented Mar 25, 2024

Re-pushed with feedback

@sftim
Copy link
Contributor

sftim commented Mar 26, 2024

@k8s-ci-robot
Copy link
Contributor

@sftim: Closed this PR.

In response to this:

/close

We'll keep https://www.k8s.dev/blog/2024/03/19/go-workspaces-in-kubernetes/

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. area/blog Issues or PRs related to the Kubernetes Blog subproject cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. language/en Issues or PRs related to English language sig/docs Categorizes an issue or PR as relevant to SIG Docs. size/L Denotes a PR that changes 100-499 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants