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

Add spotlight article on Kubernetes Contributor Summit #449

Merged
merged 2 commits into from
Nov 2, 2023

Conversation

fsmunoz
Copy link
Contributor

@fsmunoz fsmunoz commented Oct 20, 2023

Add spotlight article on the Kubernetes Contributor Summit, using KCSEU2023 as the main example.

Closes: #448

@k8s-ci-robot k8s-ci-robot added do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. labels Oct 20, 2023
@k8s-ci-robot k8s-ci-robot requested review from jberkus and jeefy October 20, 2023 13:37
@k8s-ci-robot k8s-ci-robot added the size/L Denotes a PR that changes 100-499 lines, ignoring generated files. label Oct 20, 2023
@fsmunoz
Copy link
Contributor Author

fsmunoz commented Oct 20, 2023

/cc @qedrakmar

@qedrakmar
Copy link
Member

/lgtm

@k8s-ci-robot k8s-ci-robot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Oct 20, 2023
@jberkus
Copy link
Contributor

jberkus commented Oct 23, 2023

Seems fine, unless we want to add feedback from other KCS leads?

@fsmunoz
Copy link
Contributor Author

fsmunoz commented Oct 24, 2023

@jberkus that would be good, yes. I did approach it as a 1:1 interview (similar to other spotlights), but it does have a broader scope.

On top of that, I'm also adding at least one photo from the event, from the ones in Flickr - I'm assuming there is no legal limitation with that usage.

@fsmunoz
Copy link
Contributor Author

fsmunoz commented Oct 25, 2023

@jberkus just to clarify, since after re-reading my comment I think it's a bit ambiguous: I welcome more reviewers for the content, but I wasn't aiming at changing the scope of the interview itself. We can certainly make another article with more people, even a round-table kind of thing, but I wouldn't change the structure of the current one.

@k8s-ci-robot k8s-ci-robot removed the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Oct 31, 2023
@fsmunoz
Copy link
Contributor Author

fsmunoz commented Oct 31, 2023

/retitle Add spotlight article on Kubernetes Contributor Summit

@k8s-ci-robot k8s-ci-robot changed the title WIP: Add spotlight article on Kubernetes Contributor Summit Add spotlight article on Kubernetes Contributor Summit Oct 31, 2023
@k8s-ci-robot k8s-ci-robot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Oct 31, 2023
@fsmunoz
Copy link
Contributor Author

fsmunoz commented Oct 31, 2023

I've added a group photo from the Flickr set.

@qedrakmar
Copy link
Member

/lgtm

@k8s-ci-robot k8s-ci-robot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Oct 31, 2023
@fsmunoz
Copy link
Contributor Author

fsmunoz commented Nov 1, 2023

Added kubernetes/website#43775 for website. The date is set for tomorrow, not sure if we can get approval by then - I will change the date if needed, but ideally, this would be out before KubeCon NA starts.

@jberkus
Copy link
Contributor

jberkus commented Nov 2, 2023

/approve
/lgtm

@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: fsmunoz, jberkus

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 Nov 2, 2023
@jberkus
Copy link
Contributor

jberkus commented Nov 2, 2023

Approved, that way we don't have to change the date.

@k8s-ci-robot k8s-ci-robot merged commit 5000a13 into kubernetes:master Nov 2, 2023
@sftim
Copy link
Contributor

sftim commented Nov 2, 2023

I think it's easier to line up both articles and then merge them once they're right. Maybe one day we'll have tooling to help with that.

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. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. lgtm "Looks good to me", indicates that a PR is ready to be merged. size/L Denotes a PR that changes 100-499 lines, ignoring generated files.
Projects
Development

Successfully merging this pull request may close these issues.

Add an article on organising a Kubernetes Contributor Summit
5 participants