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

fix(BA-793): Filter GQL sentinel value during endpoint rule creation #3757

Merged
merged 3 commits into from
Feb 19, 2025

Conversation

fregataa
Copy link
Member

@fregataa fregataa commented Feb 19, 2025

resolves #3756 (BA-793)

Checklist: (if applicable)

  • Milestone metadata specifying the target backport version
  • Mention to the original issue

@fregataa fregataa added the skip:changelog Make the action workflow to skip towncrier check label Feb 19, 2025
@fregataa fregataa added this to the 25Q1 milestone Feb 19, 2025
@fregataa fregataa self-assigned this Feb 19, 2025
@github-actions github-actions bot added comp:manager Related to Manager component size:S 10~30 LoC labels Feb 19, 2025
@fregataa fregataa changed the title fix: Filter GQL sentinel value during endpoint rule creation fix(BA-796): Filter GQL sentinel value during endpoint rule creation Feb 19, 2025
@fregataa fregataa changed the title fix(BA-796): Filter GQL sentinel value during endpoint rule creation fix(BA-793): Filter GQL sentinel value during endpoint rule creation Feb 19, 2025
graphite-app bot pushed a commit to lablup/backend.ai-webui that referenced this pull request Feb 19, 2025
…#3202)

Resolves #3201 (FR-555)
### TL;DR

> For test, please pull this lablup/backend.ai#3757 to your test server

Redesigned the auto-scaling rule editor interface to improve usability and add separate controls for scale up/down operations.

### What changed?

- Added distinct scale up/down options instead of combining them
- Simplified metric selection with autocomplete and predefined options
- Consolidated condition inputs (metric, comparator, threshold) into a single row
- Removed redundant min/max replica fields based on scaling direction
- Added tooltips and improved validation logic
- Updated translations for all supported languages with new UI strings

### How to test?

1. Open the endpoint detail page
2. Click to add a new auto-scaling rule
3. Verify the new UI with separate scale up/down options
4. Test metric name autocomplete functionality
5. Confirm min/max replicas show correctly based on scale direction
6. Validate form submissions work for both scale up and down scenarios

### Why make this change?

The previous interface combined scale up/down operations which made it confusing for users to configure rules correctly. This redesign separates the concerns and provides a more intuitive workflow while reducing potential configuration errors through better validation and clearer UI elements.
@fregataa fregataa added this pull request to the merge queue Feb 19, 2025
Merged via the queue into main with commit 4e3e006 Feb 19, 2025
23 checks passed
@fregataa fregataa deleted the fix/create-rule-fail branch February 19, 2025 06:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
comp:manager Related to Manager component size:S 10~30 LoC skip:changelog Make the action workflow to skip towncrier check
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Create endpoint auto scaling rule fails
1 participant