Refactoring jobs list command to align with the LLD | Add retry, environment and validation utils #6430
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
saanikaguptamicrosoft
requested review from
JeffreyCA,
hemarina,
rajeshkamal5050,
tg-msft,
vhvb1989,
wbreza and
weikanglim
as code owners
cli/azd/extensions/azure.ai.finetune/internal/providers/openai/provider.go
Show resolved
Hide resolved
cli/azd/extensions/azure.ai.finetune/internal/providers/factory/provider_factory.go
Outdated
Show resolved
Hide resolved
cli/azd/extensions/azure.ai.finetune/internal/providers/factory/provider_factory.go
Outdated
Show resolved
Hide resolved
cli/azd/extensions/azure.ai.finetune/internal/providers/factory/provider_factory.go
Outdated
Show resolved
Hide resolved
cli/azd/extensions/azure.ai.finetune/internal/providers/factory/provider_factory.go
Show resolved
Hide resolved
saanikaguptamicrosoft
changed the title
saanikaguptamicrosoft
changed the title
saanikaguptamicrosoft
changed the title
achauhan-scc
approved these changes
Dec 29, 2025
achauhan-scc
merged commit bb7f9a1
into
Azure:Model-Customization-Dev
1 of 2 checks passed
achauhan-scc
added a commit
that referenced
this pull request
Jan 10, 2026
* Moving code Azure fork (#6429) * adding draft for fine tuning * adding service target * fixing root command format * cleaning unused commands and adding operation/sub-operation commands * adding more details to command * fixes for relative path * adding registry entry * adding git download * adding 0.0.3 * fixing bug in deploy * adding printing format * re-structuring code * adding 0.0.5 * 0.0.6 * adding low level design details * fixing ext for build * adding restructured code * reverting any registry changes * Refactoring jobs list command to align with the LLD | Add retry, environment and validation utils (#6430) * initial changes * Add retry util for exponential backoff with jitter strategy * Clean-up * Standardize CLI text to lowercase conventions, and remove redundant error messages * Address comments * Add common util for fetch environment variables * Cosmetic changes * Cosmetic changes 2 * Fix runtime error * Fix error message format * Refactoring jobs show command to align with the LLD (#6434) * Create command implementation for FineTuning CLI (#6437) * Create command for ft cli * handling null pointer for state * removing changes from old converter and yaml logic * pr review changes + scheme implementation for job params * a few formatting changes * removing ofAuto value for hyperparameters * more formatting changes * adding command line parameters + formatting * minor merge fixes * Build and Release Pipelines for Finetuning CLI (#6449) * Adding pipeline scripts * Adding pipeline scripts * fixing version path * fixing build command * adding init command (#6442) * adding init command * update version * network isloation changes. (#6458) * Adding pipeline scripts * Adding pipeline scripts * fixing version path * fixing build command * Djurek/test-network-isolation (#6455) * Update 1es-redirect.yml * Trivial change to test pipeline * Show GOPROXY * Use golang internalModuleProxy --------- Co-authored-by: Daniel Jurek <djurek@microsoft.com> * Cleaning code to raise a PR to main (#6457) * adding init command * update version * cleaning some code * updating version to 0.0.7-preview (#6462) * Rename changelog.md to CHANGELOG.md * adding copyright header text * applying pr review suggestion: remove providers section from extension.yaml Co-authored-by: JeffreyCA <jeffreychen@microsoft.com> * updating version.txt to 0.0.8-preview * simplying utils for file check * reanming _uploadFile to uploadFile Co-authored-by: JeffreyCA <jeffreychen@microsoft.com> * removing duplicate import * removing unnecessary todo * Adding cspell dict * updating cspell.yaml * updating cspell.yaml * Updating finetune word in global cspell.yaml * PR Review Changes: Updating ParseGithubUrl func (#6476) * Updating parseGithubUrl func * updating resource prompt logic * copilot suggestion to remove unused variable Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * gh copilot suggestion Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> --------- Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * Updating extension name in read me file. (#6483) * adding init command * update version * cleaning some code * updating readme * resolving review comments * resolving review comments * using arm resourceId parser * Finetuning CLU - Using OSS package go-retry instead of custom logic (#6485) * Using oss go-retry package * fixing retry count * Allowing user to override the fine tuning route (#6484) * adding init command * update version * cleaning some code * updating readme * allowing fine tuning routes to be overriden by envrionment variables * spell check fix (#6486) * adding init command * update version * cleaning some code * updating readme * allowing fine tuning routes to be overriden by envrionment variables * fixing spell check issues --------- Co-authored-by: saanikaguptamicrosoft <gsaanika@gmail.com> Co-authored-by: Zubairuddin Mohammed <zmohammed@microsoft.com> Co-authored-by: Daniel Jurek <djurek@microsoft.com> Co-authored-by: JeffreyCA <jeffreychen@microsoft.com> Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Notes
Testing
Help command
Env not set
Happy case
Retry fails with wifi off
Retry succeeds on attempt 2 (Added temp changes and logs for testing)
With flags