Comment 6 for bug 1786337

Revision history for this message
Ben (docbrown-) wrote :

I have tested the bionic proposed update in the bionic docker container and can confirm the fix:

root@hillValley:/# python -c 'from phabricator import Phabricator; phab = Phabricator(host="http://127.0.0.1:9091/api/", token="api-nojs2ip33hmp4zn6u6cf72w7d6yh"); phab.diffusion.commit.search(constraints={"query": "b634a23c82be"})'
Traceback (most recent call last):
  File "<string>", line 1, in <module>
  File "/usr/lib/python2.7/dist-packages/phabricator/__init__.py", line 248, in __call__
    return self._request(**kwargs)
  File "/usr/lib/python2.7/dist-packages/phabricator/__init__.py", line 323, in _request
    data = self._parse_response(response)
  File "/usr/lib/python2.7/dist-packages/phabricator/__init__.py", line 333, in _parse_response
    raise APIError(parsed['error_code'], parsed['error_info'])
phabricator.APIError: ERR-CONDUIT-CALL: Conduit API method "commit.search" does not exist.

root@hillValley:/# python3 -c 'from phabricator import Phabricator; phab = Phabricator(host="http://127.0.0.1:9091/api/", token="api-nojs2ip33hmp4zn6u6cf72w7d6yh"); phab.diffusion.commit.search(constraints={"query": "b634a23c82be"})'
Traceback (most recent call last):
  File "<string>", line 1, in <module>
  File "/usr/lib/python3/dist-packages/phabricator/__init__.py", line 248, in __call__
    return self._request(**kwargs)
  File "/usr/lib/python3/dist-packages/phabricator/__init__.py", line 323, in _request
    data = self._parse_response(response)
  File "/usr/lib/python3/dist-packages/phabricator/__init__.py", line 333, in _parse_response
    raise APIError(parsed['error_code'], parsed['error_info'])
phabricator.APIError: ERR-CONDUIT-CALL: Conduit API method "commit.search" does not exist.

root@hillValley:/# apt-get install -y python-phabricator/bionic-proposed python3-phabricator/bionic-proposed
Reading package lists... Done
Building dependency tree
Reading state information... Done
Selected version '0.7.0-1ubuntu0.1' (Ubuntu:18.04/bionic-proposed [all]) for 'python-phabricator'
Selected version '0.7.0-1ubuntu0.1' (Ubuntu:18.04/bionic-proposed [all]) for 'python3-phabricator'
The following packages will be upgraded:
  python-phabricator python3-phabricator
2 upgraded, 0 newly installed, 0 to remove and 13 not upgraded.
Need to get 32.8 kB of archives.
After this operation, 5120 B of additional disk space will be used.
Get:1 http://archive.ubuntu.com/ubuntu bionic-proposed/universe amd64 python-phabricator all 0.7.0-1ubuntu0.1 [16.1 kB]
Get:2 http://archive.ubuntu.com/ubuntu bionic-proposed/universe amd64 python3-phabricator all 0.7.0-1ubuntu0.1 [16.7 kB]
Fetched 32.8 kB in 1s (22.2 kB/s)
debconf: delaying package configuration, since apt-utils is not installed
(Reading database ... 7544 files and directories currently installed.)
Preparing to unpack .../python-phabricator_0.7.0-1ubuntu0.1_all.deb ...
Unpacking python-phabricator (0.7.0-1ubuntu0.1) over (0.7.0-1) ...
Preparing to unpack .../python3-phabricator_0.7.0-1ubuntu0.1_all.deb ...
Unpacking python3-phabricator (0.7.0-1ubuntu0.1) over (0.7.0-1) ...
Setting up python-phabricator (0.7.0-1ubuntu0.1) ...
Setting up python3-phabricator (0.7.0-1ubuntu0.1) ...

root@hillValley:/# python -c 'from phabricator import Phabricator; phab = Phabricator(host="http://127.0.0.1:9091/api/", token="api-nojs2ip33hmp4zn6u6cf72w7d6yh"); phab.diffusion.commit.search(constraints={"query": "b634a23c82be"})'

root@hillValley:/# python3 -c 'from phabricator import Phabricator; phab = Phabricator(host="http://127.0.0.1:9091/api/", token="api-nojs2ip33hmp4zn6u6cf72w7d6yh"); phab.diffusion.commit.search(constraints={"query": "b634a23c82be"})'