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-67): service list CLI command always print image as (null) #3017

Draft
wants to merge 4 commits into
base: feat/add-canonical-field-to-image-gql-api
Choose a base branch
from

Conversation

jopemachine
Copy link
Member

@jopemachine jopemachine commented Nov 4, 2024

Fixes #3014 (BA-67).

Test

Tested manually.

./backend.ai service list
Endpoint Id                           Name                    Image                                             Desired Session Count  Routings                     Session Owner                         Open To Public
------------------------------------  ----------------------  ----------------------------------------------  -----------------------  ---------------------------  ------------------------------------  ----------------
8e2175fb-1a07-4bb8-8a8b-af0d0419e111  bai-serve-autumn79      cr.backend.ai/multiarch/python:3.9-ubuntu20.04                        2  PROVISIONING 1, UNHEALTHY 1  f38dea23-50fa-42a0-b5ae-338f5f4693f4  False
f72c3527-7996-413f-bd90-f5b037dfbf92  bai-serve-manuelmorris  cr.backend.ai/multiarch/python:3.9-ubuntu20.04                        2  UNHEALTHY 2                  f38dea23-50fa-42a0-b5ae-338f5f4693f4  False
62174018-06af-4d25-9acc-2d75aec28d58  bai-serve-steven63      cr.backend.ai/multiarch/python:3.9-ubuntu20.04                        2  UNHEALTHY 2                  f38dea23-50fa-42a0-b5ae-338f5f4693f4  False

Checklist: (if applicable)

  • Milestone metadata specifying the target backport version
  • Test case(s) to:
    • Demonstrate the difference of before/after

Copy link
Member Author

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @jopemachine and the rest of your teammates on Graphite Graphite

@github-actions github-actions bot added comp:client Related to Client component comp:cli Related to CLI component size:XS ~10 LoC labels Nov 4, 2024
@jopemachine jopemachine added the type:bug Reports about that are not working label Nov 4, 2024
@jopemachine jopemachine added this to the 24.09 milestone Nov 4, 2024
@jopemachine jopemachine force-pushed the feat/add-canonical-field-to-image-gql-api branch from 99b8498 to b53e75d Compare November 4, 2024 02:40
@jopemachine jopemachine force-pushed the topic/11-04-fix_service_list_cli_command_always_print_image_as_null_ branch 4 times, most recently from 4fb985e to 14a3c60 Compare November 4, 2024 08:16
@jopemachine jopemachine modified the milestones: 24.09, 24.12 Nov 4, 2024
@jopemachine jopemachine force-pushed the feat/add-canonical-field-to-image-gql-api branch from 1c01a38 to 1abfce2 Compare November 4, 2024 09:27
@jopemachine jopemachine force-pushed the topic/11-04-fix_service_list_cli_command_always_print_image_as_null_ branch from 14a3c60 to 3d7681d Compare November 4, 2024 09:28
@jopemachine jopemachine linked an issue Nov 24, 2024 that may be closed by this pull request
@jopemachine jopemachine changed the title fix: service list CLI command always print image as (null) fix(BA-67): service list CLI command always print image as (null) Jan 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
comp:cli Related to CLI component comp:client Related to Client component size:XS ~10 LoC type:bug Reports about that are not working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

service list CLI command print Image as (null)
1 participant