servo / saltfs

Salt Stack Filesystem
Apache License 2.0
56 stars 107 forks source link

Add hyper-serde to homu. #907

Closed jdm closed 5 years ago

jdm commented 6 years ago

This change is Reviewable

jdm commented 6 years ago

There's something weird about this repository, possibly because it was transferred:

Traceback (most recent call last):
  File "/home/servo/homu/_venv/bin/bottle.py", line 862, in _handle
    return route.call(**args)
  File "/home/servo/homu/_venv/bin/bottle.py", line 1740, in wrapper
    rv = callback(*a, **ka)
  File "/home/servo/homu/_venv/lib/python3.4/site-packages/homu/server.py", line 207, in callback
    repo = get_repo(repo_label, repo_cfg)
  File "/home/servo/homu/_venv/lib/python3.4/site-packages/homu/server.py", line 58, in get_repo
    assert repo.owner.login == repo_cfg['owner']
AttributeError: 'NoneType' object has no attribute 'owner'
Traceback (most recent call last):
  File "/home/servo/homu/_venv/bin/bottle.py", line 862, in _handle
    return route.call(**args)
  File "/home/servo/homu/_venv/bin/bottle.py", line 1740, in wrapper
    rv = callback(*a, **ka)
  File "/home/servo/homu/_venv/lib/python3.4/site-packages/homu/server.py", line 116, in queue
    if repo_label in g.repos and g.repos[repo_label].treeclosed >= 0:
AttributeError: 'NoneType' object has no attribute 'treeclosed'
bors-servo commented 5 years ago

:umbrella: The latest upstream changes (presumably #926) made this pull request unmergeable. Please resolve the merge conflicts.

nox commented 5 years ago

It's an underscore, not an hyphen. :)

jdm commented 5 years ago

@bors-servo r+

bors-servo commented 5 years ago

:pushpin: Commit 8016af4 has been approved by jdm

bors-servo commented 5 years ago

:hourglass: Testing commit 8016af45775ff1c0b59ee81d656a3495fc3bbee3 with merge 15ab267064b5c3b3e74529e8f3a22507d9bda959...

bors-servo commented 5 years ago

:sunny: Test successful - checks-travis Approved by: jdm Pushing 15ab267064b5c3b3e74529e8f3a22507d9bda959 to master...

jdm commented 5 years ago
Traceback (most recent call last):
  File "/home/servo/homu/_venv/bin/bottle.py", line 862, in _handle
    return route.call(**args)
  File "/home/servo/homu/_venv/bin/bottle.py", line 1740, in wrapper
    rv = callback(*a, **ka)
  File "/home/servo/homu/_venv/lib/python3.4/site-packages/homu/server.py", line 310, in github
    repo_label = g.repo_labels[owner, info['repository']['name']]
KeyError: ('servo', 'hyper_serde')

That's from trying to r+ something.

Traceback (most recent call last):
  File "/home/servo/homu/_venv/bin/bottle.py", line 862, in _handle
    return route.call(**args)
  File "/home/servo/homu/_venv/bin/bottle.py", line 1740, in wrapper
    rv = callback(*a, **ka)
  File "/home/servo/homu/_venv/lib/python3.4/site-packages/homu/server.py", line 119, in queue
    g.cfg['repo'][repo_label]['owner'],
KeyError: 'hyper_serde'

That's from trying to visit https://build.servo.org/homu/queue/hyper_serde.

jdm commented 5 years ago

I deployed the wrong version; now https://build.servo.org/homu/queue/hyper_serde works but I can't make it synchronize.