#157 Improve the output of freshmaker-cli event details.
Merged by jkaluza. Opened by jkaluza.
jkaluza/freshmaker cli-detailed-events  into  master

Download 157.patch

This PR changes the output of freshmaker-cli [event_id]:

  • The Original NVR and Rebuilt NVR column are replaced by Name, Version-Release and Rebuilt Version-Release columns. It does not repeat the Name in both columns that way and therefore the rows in resulting tables are shorter.
  • It sorts the rows based on the artifact build dependency and use sort of indentation, so you can see from the output dependencies between artifacts. I'm not saying that's ideal, but I'm open to further PRs changing this :).

Example output now:

$ ./freshmaker-cli --redhat 356
Getting Freshmaker event id 356
Event id: 356
Build ID       Task ID  State    Name                 Version-Release    Rebuilt Version-Release      State Reason
-----------  ---------  -------  -------------------  -----------------  ---------------------------  -------------------
-->      70   14745916  DONE     foo-parent-docker    v3.6.173.0.49-5    v3.6.173.0.49-5.1513169992   Built successfully.
---->    80   14746332  DONE     foo2-docker          2.0-4              2.0-4.1513170031             Built successfully.
---->    78   14746330  DONE     foo3-docker          v3.6.173.0.49-6    v3.6.173.0.49-6.1513170023   Built successfully.
---->    79   14746331  DONE     foo4-docker          v3.6.173.0.49-5    v3.6.173.0.49-5.1513170027   Built successfully.
-->      65   14745907  DONE     foo5-parent-docker   v3.6.173.0.63-11   v3.6.173.0.63-11.1513169971  Built successfully.
---->    83   14746394  DONE     foo6-docker          v3.6.173.0.63-11   v3.6.173.0.63-11.1513170044  Built successfully.
---->    76   14746392  DONE     foo7-docker          v3.6.173.0.63-11   v3.6.173.0.63-11.1513170015  Built successfully.

Looks good for me.

In addition, probably State Reason could be avoided to display in this table for saving much space in screen. Instead, we may add another option or command to display a task's detailed information that may be similar with koji taskinfo.

It's fine to merge this PR.

Pull-Request has been merged by jkaluza

Metadata