-
Notifications
You must be signed in to change notification settings - Fork 70
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 deployment inspect and list commands #1544
Conversation
@@ -246,6 +245,10 @@ func getDeploymentConfig(coreDeploymentPointer *astroplatformcore.Deployment, pl | |||
if coreDeployment.SchedulerAu != nil { | |||
deploymentMap["scheduler_au"] = *coreDeployment.SchedulerAu | |||
} | |||
if coreDeployment.CloudProvider != nil { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we need to change condition on line 232 for region
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yes I think you are right
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #1544 +/- ##
==========================================
+ Coverage 86.06% 86.11% +0.05%
==========================================
Files 112 112
Lines 14916 14919 +3
==========================================
+ Hits 12837 12848 +11
+ Misses 1258 1252 -6
+ Partials 821 819 -2 ☔ View full report in Codecov by Sentry. |
* fix deployment inspect and list commands * fix lint * fix test * fix test * update test * fix lint
Description
🎟 Issue(s)
Related #XXX
🧪 Functional Testing
📸 Screenshots
📋 Checklist
make test
before taking out of draftmake lint
before taking out of draft