feat(template): expose latest job id in template list across clients#546
feat(template): expose latest job id in template list across clients#546xiaojunxiang2023 wants to merge 1 commit into
Conversation
51a2cd3 to
bfa5c81
Compare
|
Thanks for this proposal @xiaojunxiang2023 — exposing the latest job ID in template list responses is a great idea. It closes a real UX gap where operators had to guess the right However, it looks like this PR now has conflicts with Could you please rebase this branch onto the latest Happy to take another look once it's rebased. Thanks for the contribution! 🙏 |
Return each template's latest create/rebuild job id from CubeMaster list and detail APIs so operators can inspect build status without guessing ids. Propagate the field through CubeAPI, cubemastercli, web UI, and SDKs. Signed-off-by: xiaojunxiang <xiaojunxiang@kingsoft.com>
bfa5c81 to
599113a
Compare
Thanks for the review! I've rebased onto the latest master and resolved the conflicts in template.go, httpservice/cube/template.go, and store.go. For tpl ls, I kept the JOB_ID column in the existing tabwriter output — on current master, the TUI changes from #580 apply to watch/status, while ls still uses the table layout, so this should fit without extra rework. API, SDK, and Web UI parts are unchanged. Ready for another look whenever you have time. |
Summary
Template list/detail responses now include each template's latest create/rebuild job id (
job_id/jobID). This lets operators look up build status without guessing ids aftertpl ls, Web UI browsing, or SDK listing.Fixes the gap where
cubemastercli tpl lsshowed building template ids but not the corresponding job id needed fortpl status --job-id/tpl watch --job-id.Changes
job_idTemplateImageJobper templateJOB_IDcolumntpl ls/tpl ls -o wide;tpl infoprintsjob_idwhen presentjobIDGET /templatesandGET /templates/:idjobIDTemplateInfo.job_idTemplateInfo.JobIDSemantics: returns the latest job id for the template (including
READY/FAILED), not only in-progress jobs. Templates with no job history (e.g. legacy entries) omit the field.Example
Deploy and Verify