fix(hubio): when error occurs, exit code should be set to non-zero

[mapleeit]

Fix that when error occurs, the exit code still be 0. It makes tests confusing and hard to detect whether it's successful or not.

❯ JINA_HUBBLE_REGISTRY=http://localhost:3000 jina hub push case9; echo $?

MMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMWWWMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMM
MMMMMMMMMMMMMMMMMMMMMMMMMMMMMWNNNNNNNWMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMM
MMMMMMMMMMMMMMMMMMMMMMMMMMMMMNNNNNNNNNMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMM
MMMMMMMMMMMMMMMMMMMMMMMMMMMMMWNNNNNNNNMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMM
MMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMWNNNWWMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMM
MMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMM
MMMMMMMMMMMMWxxxxxxxxxOMMMMMNxxxxxxxxx0MMMMMKddddddxkKWMMMMMMMMMMMMXOxdddxONMMMM
MMMMMMMMMMMMXllllllllldMMMMM0lllllllllxMMMMMOllllllllllo0MMMMMMMM0olllllllllo0MM
MMMMMMMMMMMMXllllllllldMMMMM0lllllllllxMMMMMOlllllllllllloWMMMMMdllllllllllllldM
MMMMMMMMMMMMXllllllllldMMMMM0lllllllllxMMMMMOllllllllllllloMMMM0lllllllllllllllK
MMMMMMMMMMMMKllllllllldMMMMM0lllllllllxMMMMMOllllllllllllllKMMM0lllllllllllllllO
MMMMMMMMMMMMKllllllllldMMMMM0lllllllllxMMMMMOllllllllllllll0MMMMollllllllllllllO
MWOkkkkk0MMMKlllllllllkMMMMM0lllllllllxMMMMMOllllllllllllll0MMMMMxlllllllllllllO
NkkkkkkkkkMMKlllllllloMMMMMM0lllllllllxMMMMMOllllllllllllll0MMMMMMWOdolllllllllO
KkkkkkkkkkNMKllllllldMMMMMMMMWWWWWWWWWMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMM
MOkkkkkkk0MMKllllldXMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMM
MMWX00KXMMMMXxk0XMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMMM

▶️  /Users/miaozhaofeng/.pyenv/versions/3.9.5/bin/jina hub push case9
🔧️                            cli = hub
                            force = None
🔧️                            hub = push
🔧️                       no-usage = False
🔧️                           path = case9
                           secret = None

          HubIO@3649[E]:Error while pushing session_id=3443c17e-deef-11eb-86e5-1e00d92e5291:
Exception('Unknown Error')
0

[codecov]

Codecov Report

Merging #2871 (d613a78) into master (0094bbf) will increase coverage by 6.60%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #2871      +/-   ##
==========================================
+ Coverage   82.21%   88.82%   +6.60%     
==========================================
  Files         106      138      +32     
  Lines        7064     9515    +2451     
==========================================
+ Hits         5808     8452    +2644     
+ Misses       1256     1063     -193     

Flag	Coverage Δ
daemon	43.40% <ø> (?)
jina	88.77% <ø> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files	Coverage Δ
jina/__init__.py	71.64% <ø> (ø)
jina/checker.py	96.55% <ø> (ø)
jina/clients/__init__.py	100.00% <ø> (ø)
jina/clients/base/__init__.py	90.90% <ø> (ø)
jina/clients/base/grpc.py	63.82% <ø> (ø)
jina/clients/base/http.py	93.18% <ø> (ø)
jina/clients/base/websocket.py	86.36% <ø> (ø)
jina/clients/grpc.py	100.00% <ø> (ø)
jina/clients/helper.py	100.00% <ø> (ø)
jina/clients/http.py	100.00% <ø> (ø)
... and 215 more

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 14cbb0d...d613a78. Read the comment docs.

[hanxiao]

hub test is broken

[github-actions]

Latency summary

Current PR yields:

• 😶 index QPS at 1345, delta to last 1 avg.: -2%
• 😶 query QPS at 24, delta to last 1 avg.: -5%
• 😶 import jina within 0.2558s, delta to last 1 avg.: +1%

Breakdown

Version	Index QPS	Query QPS	Import Time (s)
current	1345	24	0.2558
2.0.4	1379	25	0.2515

Backed by latency-tracking. Further commits will update this comment.

[mapleeit]

Ready to be reviewed again. @hanxiao @numb3r3

[hanxiao]

just to clarify, this line assumes that push is always used under CLI, and it will invalidate the usage from other Python functions.