Skip to content

Conversation

@shiyasmohd
Copy link
Contributor

  • Add Glob pattern support for worker node job affinity
  • Update admin-api and ampctl to accept glob pattern
  • Added unit tests.

Closes #1508

@shiyasmohd shiyasmohd self-assigned this Dec 22, 2025
Copy link
Contributor

@LNSD LNSD left a comment

Choose a reason for hiding this comment

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

Please, check my comments 🙂

@shiyasmohd shiyasmohd force-pushed the shiyasmohd/node-id-glob branch from 4ae7737 to af76224 Compare December 22, 2025 18:55
@shiyasmohd
Copy link
Contributor Author

@LNSD All comments has been solved now.

Copy link
Contributor

@LNSD LNSD left a comment

Choose a reason for hiding this comment

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

In general, it's ok, I miss the docs.

JFYI: You can ask Claude to follow the .patterns/error-reporting.md and .patterns/documentation.md guidelines.

@shiyasmohd shiyasmohd force-pushed the shiyasmohd/node-id-glob branch from af76224 to df6c756 Compare December 23, 2025 15:41
@shiyasmohd shiyasmohd force-pushed the shiyasmohd/node-id-glob branch from df6c756 to 998dc72 Compare December 23, 2025 16:08
@shiyasmohd
Copy link
Contributor Author

@LNSD Pushed the changes.

@shiyasmohd shiyasmohd requested a review from LNSD December 23, 2025 16:19
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.

Glob based worker node job affinity

3 participants