Skip to content

Create cloud-compute zulip user group #1970

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.

Sign up for GitHub

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

Jump to bottom
Draft
wants to merge 3 commits into
base: master
Choose a base branch
from
Draft

Create cloud-compute zulip user group #1970

wants to merge 3 commits into from

Conversation

jieyouxu
Copy link
Member

This is needed to configure the infra announcement channel so the zulip user group membership is synced automatically with marker team membership.

Accompanies #1969.

See #t-infra > Communication method for dev desktops @ 💬 for more details.

@jieyouxu jieyouxu added needs-team-repo-admin-review This change requires one of the `team-repo-admins` to review. S-waiting-on-review Status: waiting on review from a team/WG/PG lead, an infra-admin, and/or a team-repo-admin. T-infra Relevant to the infrastructure team. S-blocked Status: blocked and removed S-waiting-on-review Status: waiting on review from a team/WG/PG lead, an infra-admin, and/or a team-repo-admin. labels Aug 30, 2025
@jieyouxu
Copy link
Member Author 

[ERROR rust_team::validate] validation error: person `Kivooeo` in 'cloud-compute' is a member of a Zulip user group but has no Zulip id
[ERROR rust_team::validate] validation error: person `asquared31415` in 'cloud-compute' is a member of a Zulip user group but has no Zulip id

@jieyouxu jieyouxu marked this pull request as draft August 30, 2025 05:08
@jieyouxu
Create cloud-compute zulip user group
8e40e42 
This is needed to configure the infra announcement channel so the zulip
user group membership is synced automatically with marker team
membership.
@jieyouxu jieyouxu force-pushed the infra-announcement-cloud-compute branch from 7af92c1 to 56dbe8b Compare September 6, 2025 06:30
@jieyouxu jieyouxu force-pushed the infra-announcement-cloud-compute branch 4 times, most recently from 67a5d27 to ebe9998 Compare September 6, 2025 06:54
@jieyouxu jieyouxu force-pushed the infra-announcement-cloud-compute branch from ebe9998 to 12f84a5 Compare September 6, 2025 07:17
@github-actions GitHub Actions
Copy link

github-actions bot commented Sep 6, 2025

Dry-run check results 

[WARN  sync_team] sync-team is running in dry mode, no changes will be applied.
[INFO  sync_team] synchronizing github
[INFO  sync_team] 💻 Team Diffs:
    📝 Editing team 'rust-lang/all':
      Adding member 'graciegregory' with member role
    📝 Editing team 'rust-lang/community-survey':
      Adding member 'graciegregory' with member role
    📝 Editing team 'rust-lang/wg-triage':
      Adding member 'edmilsonefs' with member role

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Mark-Simulacrum Mark-Simulacrum Awaiting requested review from Mark-Simulacrum Mark-Simulacrum will be requested when the pull request is marked ready for review Mark-Simulacrum is a code owner

@pietroalbini pietroalbini Awaiting requested review from pietroalbini pietroalbini will be requested when the pull request is marked ready for review pietroalbini is a code owner

@jdno jdno Awaiting requested review from jdno jdno will be requested when the pull request is marked ready for review jdno is a code owner

@marcoieni marcoieni Awaiting requested review from marcoieni marcoieni will be requested when the pull request is marked ready for review marcoieni is a code owner

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
needs-team-repo-admin-review This change requires one of the `team-repo-admins` to review. S-blocked Status: blocked T-infra Relevant to the infrastructure team.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@jieyouxu