Skip to content

Add sig-autoscaling slack config, WAS channel#8972

Open
jackfrancis wants to merge 3 commits intokubernetes:mainfrom
jackfrancis:sig-autoscaling-slack-was
Open

Add sig-autoscaling slack config, WAS channel#8972
jackfrancis wants to merge 3 commits intokubernetes:mainfrom
jackfrancis:sig-autoscaling-slack-was

Conversation

@jackfrancis
Copy link
Copy Markdown
Contributor

Which issue(s) this PR fixes:

This PR adds a dedicated sig-autoscaling config file for slack, and adds a new "scheduler-driven-autoscaling" channel to coordinate Workload Aware Scheduling-downstream efforts.

@k8s-ci-robot k8s-ci-robot added cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. size/S Denotes a PR that changes 10-29 lines, ignoring generated files. labels Apr 28, 2026
@k8s-ci-robot k8s-ci-robot requested a review from jeefy April 28, 2026 21:57
@k8s-ci-robot k8s-ci-robot requested a review from JoelSpeed April 28, 2026 21:57
@k8s-ci-robot k8s-ci-robot added area/slack-management Issues or PRs related to the Slack Management subproject sig/contributor-experience Categorizes an issue or PR as relevant to SIG Contributor Experience. labels Apr 28, 2026
@jackfrancis
Copy link
Copy Markdown
Contributor Author

/assign @towca @palnabarun

Comment thread communication/slack-config/sig-autoscaling/config.yaml
@jackfrancis jackfrancis force-pushed the sig-autoscaling-slack-was branch from 14245e6 to 47001d6 Compare April 29, 2026 16:03
@mrbobbytables
Copy link
Copy Markdown
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 May 6, 2026
@ameukam
Copy link
Copy Markdown
Member

ameukam commented May 6, 2026

cc @jberkus @Priyankasaggu11929

For approval. 🙏🏾

Comment thread communication/slack-config/sig-autoscaling/config.yaml Outdated
- name: sig-auth-triage
- name: sig-autoscaling
# sig-autoscaling channels are defined in sig-autoscaling/
- name: sig-autoscaling-api
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Why is this channel not going under sig-autoscaling?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Thanks for catching this.

I actually think we should retire this channel. I wasn't aware of it and reviewing it there is no activity in almost 2 years, and very little since 2023. It looks like it was active prior to 2022, but I'm no aware there is any need or a distinct channel going forward.

Image

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Will deleting this line have an "infrastructure as code" equivalent outcome of deleting the channel from k8s slack?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

No, you need to mark it as archived. See other examples in the main file of archived channels.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Thank you, I did that in the newly created communication/slack-config/sig-autoscaling/config.yaml, I assume that the same structure works there.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Perhaps I need to add a regex for the sig-autoscaling-api string to the restrictions file, I was hoping to minimize cruft, lemme know if we need that as well, thank you!

@k8s-ci-robot
Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: jackfrancis
Once this PR has been reviewed and has the lgtm label, please ask for approval from mrbobbytables. For more information see the Code Review Process.

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

Details 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 needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label May 6, 2026
Signed-off-by: Jack Francis <jackfrancis@gmail.com>
Signed-off-by: Jack Francis <jackfrancis@gmail.com>
@jackfrancis jackfrancis force-pushed the sig-autoscaling-slack-was branch from 47001d6 to 4dd4020 Compare May 6, 2026 19:32
@k8s-ci-robot k8s-ci-robot removed the lgtm "Looks good to me", indicates that a PR is ready to be merged. label May 6, 2026
@k8s-ci-robot
Copy link
Copy Markdown
Contributor

New changes are detected. LGTM label has been removed.

@k8s-ci-robot k8s-ci-robot removed the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label May 6, 2026
Signed-off-by: Jack Francis <jackfrancis@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/community-management area/slack-management Issues or PRs related to the Slack Management subproject cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. sig/contributor-experience Categorizes an issue or PR as relevant to SIG Contributor Experience. size/S Denotes a PR that changes 10-29 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

9 participants