Skip to content

chore: backport 3/5/2026#1265

Merged
weng271190436 merged 5 commits intoAzure:mainfrom
weng271190436:weiweng/backport-20260305
Mar 6, 2026
Merged

chore: backport 3/5/2026#1265
weng271190436 merged 5 commits intoAzure:mainfrom
weng271190436:weiweng/backport-20260305

Conversation

@weng271190436
Copy link
Member

@weng271190436 weng271190436 commented Mar 5, 2026

Description of your changes

b625d19 fix: correct pre-selected to preselected in scheduler framework test (#492)
b3e2321 feat: new selection mode so that CRP can select some namespace resources (#452)

I have:

  • Run make reviewable to ensure this PR is ready for review.

How has this code been tested

Special notes for your reviewer

weng271190436 and others added 3 commits March 5, 2026 11:08
…ces (Azure#452)

* add selection mode that places namespace and namespace scoped resources

Signed-off-by: Wei Weng <Wei.Weng@microsoft.com>

* remove namespace selector must be 1st restriction

Signed-off-by: Wei Weng <Wei.Weng@microsoft.com>

* use CEL validation

Signed-off-by: Wei Weng <Wei.Weng@microsoft.com>

* try double quote

Signed-off-by: Wei Weng <Wei.Weng@microsoft.com>

* simplify if else

Signed-off-by: Wei Weng <Wei.Weng@microsoft.com>

---------

Signed-off-by: Wei Weng <Wei.Weng@microsoft.com>
Co-authored-by: Wei Weng <Wei.Weng@microsoft.com>
Signed-off-by: Simon Waight <simon.waight@microsoft.com>
@weng271190436 weng271190436 changed the title Weiweng/backport 20260305 chore: backport 3/5/2026 Mar 5, 2026
ytimocin
ytimocin previously approved these changes Mar 6, 2026
Copy link
Contributor

@ytimocin ytimocin left a comment

Choose a reason for hiding this comment

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

My changes look good.

Signed-off-by: Wei Weng <Wei.Weng@microsoft.com>
@weng271190436 weng271190436 merged commit 938a947 into Azure:main Mar 6, 2026
24 of 25 checks passed
@weng271190436 weng271190436 deleted the weiweng/backport-20260305 branch March 6, 2026 18:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants