Skip to content
Snippets Groups Projects

Indicate where artifacts are going to and coming from in the log

Merged Qinusty requested to merge Qinusty/275 into master

Description

Indicate where artifacts are coming from and being pushed to in the logs.

Changes proposed in this merge request:

  • Make Element.get_brief_display_key() public for logging purposes of other modules
  • Add info messages to indicate the remote being used for pushing to a cache
  • Add info messages to indicate the remote being used for pulling from a cache

This merge request, when approved, will close issue/bug: #275 (closed)

CHANGELOG/Release Notes

  • Element.get_brief_display_key() is now publicly accessible.

Preview

[--:--:--][72f8f716][ pull:hello.bst                     ] START   CAS-interaction/hello/72f8f716-pull.511.log
[--:--:--][72f8f716][ pull:hello.bst                     ] INFO    Pulling 72f8f716 <- remote (https://localhost:9999)
[--:--:--][72f8f716][ pull:hello.bst                     ] INFO    Downloaded artifact 72f8f716
[00:00:00][72f8f716][ pull:hello.bst                     ] SUCCESS CAS-interaction/hello/72f8f716-pull.511.log

...

[--:--:--][72f8f716][ push:hello.bst                     ] START   CAS-interaction/hello/72f8f716-push.1010.log
[--:--:--][72f8f716][ push:hello.bst                     ] START   Pushing artifact 72f8f716
[--:--:--][72f8f716][ push:hello.bst                     ] INFO    Pushing 72f8f716 -> https://localhost:9999
[--:--:--][72f8f716][ push:hello.bst                     ] INFO    Pushed artifact 72f8f716
[00:00:00][72f8f716][ push:hello.bst                     ] SUCCESS Pushing artifact 72f8f716
[00:00:00][72f8f716][ push:hello.bst                     ] SUCCESS CAS-interaction/hello/72f8f716-push.1010.log
Edited by Qinusty

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Qinusty resolved all discussions

    resolved all discussions

  • Qinusty added 3 commits

    added 3 commits

    • 3b16173a - element.py: Make Element._get_brief_display_key() available to Internal API
    • 6dd5f927 - cascache.py: Add push info detailing which remote is being pushed to
    • 7909594e - cascache.py: Add pull info detailing which remote is being pulled from

    Compare with previous version

  • Qinusty marked as a Work In Progress

    marked as a Work In Progress

  • Qinusty added 3 commits

    added 3 commits

    • 153c3cc2 - element.py: Make Element._get_brief_display_key() available to Internal API
    • 74828141 - cascache.py: Add push info detailing which remote is being pushed to
    • 893e699e - cascache.py: Add pull info detailing which remote is being pulled from

    Compare with previous version

  • Qinusty added 1 commit

    added 1 commit

    • 56e8aa46 - pylint: Remove trailing whitespace

    Compare with previous version

  • Qinusty mentioned in issue #481 (closed)

    mentioned in issue #481 (closed)

  • Qinusty added 1 commit

    added 1 commit

    • b483beb8 - pylint: Remove trailing whitespace

    Compare with previous version

  • Qinusty unmarked as a Work In Progress

    unmarked as a Work In Progress

  • Tristan Van Berkom
  • Overall this looks fine, please make sure the commit "pylint: Remove trailing whitespace" goes away.

    That commit simply fixes problems which occurred on your branch, as such it should not be added to master.

    Instead, please rebase the branch so that the relevant whitespace errors are fixed directly in the commits proposed for master.

  • Qinusty resolved all discussions

    resolved all discussions

  • Qinusty added 3 commits

    added 3 commits

    • c956bb61 - element.py: Make Element._get_brief_display_key() available to Internal API
    • a876fc95 - cascache.py: Add push info detailing which remote is being pushed to
    • 17a4ede9 - cascache.py: Add pull info detailing which remote is being pulled from

    Compare with previous version

  • Qinusty added 17 commits

    added 17 commits

    • 17a4ede9...c7124c3d - 14 commits from branch master
    • fc3ba59f - element.py: Make Element._get_brief_display_key() available to Internal API
    • 01da1d19 - cascache.py: Add push info detailing which remote is being pushed to
    • a1c2f656 - cascache.py: Add pull info detailing which remote is being pulled from

    Compare with previous version

  • added 5 commits

    • a1c2f656...fe0df616 - 2 commits from branch master
    • 6559db4c - element.py: Make Element._get_brief_display_key() available to Internal API
    • 6b82e476 - cascache.py: Add push info detailing which remote is being pushed to
    • 1d36df25 - cascache.py: Add pull info detailing which remote is being pulled from

    Compare with previous version

  • Tristan Van Berkom enabled an automatic merge when the pipeline for 1d36df25 succeeds

    enabled an automatic merge when the pipeline for 1d36df25 succeeds

  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Please register or sign in to reply
    Loading