Merge lp://staging/~pwlars/core-selftest-agent/agent-config into lp://staging/core-selftest-agent

Proposed by Paul Larson
Status: Merged
Approved by: Paul Larson
Approved revision: 9
Merged at revision: 5
Proposed branch: lp://staging/~pwlars/core-selftest-agent/agent-config
Merge into: lp://staging/core-selftest-agent
Diff against target: 126 lines (+95/-1)
4 files modified
called-by-tarmac.py (+6/-0)
core_selftest_agent/__init__.py (+45/-1)
core_selftest_agent/constants.py (+42/-0)
requirements.txt (+2/-0)
To merge this branch: bzr merge lp://staging/~pwlars/core-selftest-agent/agent-config
Reviewer Review Type Date Requested Status
Para Siva (community) Approve
Paul Larson Needs Resubmitting
Francis Ginther Approve
Review via email: mp+259799@code.staging.launchpad.net

Commit message

Add config bits

Description of the change

Add config bits

To post a comment you must log in.
Revision history for this message
Francis Ginther (fginther) wrote :

A have a couple of inline comments. I think we're now safe to remove the modified hostname methods.

review: Needs Fixing
Revision history for this message
Paul Larson (pwlars) wrote :

Actually, I don't know that I'll even need a retry queue so I took it out for now. It may be necessary to retry internally for a failed manifest download or something, but I don't think this service has a concept of an infrastructure failure outside of its control that would require requeueing. I also replaced the hostname bit with socket.gethostname()

review: Needs Resubmitting
Revision history for this message
Francis Ginther (fginther) wrote :

I'm happy with this. As mentioned over IRC, there really is no reason for the agent to retry messages. The input is a fan-out exchange so every consumer will already have a copy, no need to throw it back to see if another consumer can make use of it. I also doubt we'll need deadletter, but am happy to wait and see on this.

I'm curious as to why you kept ".replace('-machine', '')" on the hostname. I don't think this is hurting anything. But I'm not going to block on this either.

review: Approve
Revision history for this message
Ubuntu CI Bot (uci-bot) wrote :
Download full text (3.4 KiB)

The attempt to merge lp:~pwlars/core-selftest-agent/agent-config into lp:core-selftest-agent failed. Below is the output from the failed tests.

Using base prefix '/usr'
New python executable in /tmp/venv-core-selftest-agentwtd8oky_/bin/python3
Also creating executable in /tmp/venv-core-selftest-agentwtd8oky_/bin/python
Installing setuptools, pip...done.
Running virtualenv with interpreter /usr/bin/python3
Ignoring indexes: https://pypi.python.org/simple/
Downloading/unpacking python-logstash==0.4.2 (from -r requirements.txt (line 1))
Cleaning up...
Exception:
Traceback (most recent call last):
  File "/tmp/venv-core-selftest-agentwtd8oky_/lib/python3.4/site-packages/pip/basecommand.py", line 122, in main
    status = self.run(options, args)
  File "/tmp/venv-core-selftest-agentwtd8oky_/lib/python3.4/site-packages/pip/commands/install.py", line 278, in run
    requirement_set.prepare_files(finder, force_root_egg_info=self.bundle, bundle=self.bundle)
  File "/tmp/venv-core-selftest-agentwtd8oky_/lib/python3.4/site-packages/pip/req.py", line 1177, in prepare_files
    url = finder.find_requirement(req_to_install, upgrade=self.upgrade)
  File "/tmp/venv-core-selftest-agentwtd8oky_/lib/python3.4/site-packages/pip/index.py", line 252, in find_requirement
    for page in self._get_pages(locations, req):
  File "/tmp/venv-core-selftest-agentwtd8oky_/lib/python3.4/site-packages/pip/index.py", line 387, in _get_pages
    page = self._get_page(location, req)
  File "/tmp/venv-core-selftest-agentwtd8oky_/lib/python3.4/site-packages/pip/index.py", line 568, in _get_page
    session=self.session,
  File "/tmp/venv-core-selftest-agentwtd8oky_/lib/python3.4/site-packages/pip/index.py", line 670, in get_page
    resp = session.get(url, headers={"Accept": "text/html"})
  File "/tmp/venv-core-selftest-agentwtd8oky_/lib/python3.4/site-packages/pip/_vendor/requests/sessions.py", line 395, in get
    return self.request('GET', url, **kwargs)
  File "/tmp/venv-core-selftest-agentwtd8oky_/lib/python3.4/site-packages/pip/download.py", line 237, in request
    return super(PipSession, self).request(method, url, *args, **kwargs)
  File "/tmp/venv-core-selftest-agentwtd8oky_/lib/python3.4/site-packages/pip/_vendor/requests/sessions.py", line 349, in request
    prep = self.prepare_request(req)
  File "/tmp/venv-core-selftest-agentwtd8oky_/lib/python3.4/site-packages/pip/_vendor/requests/sessions.py", line 287, in prepare_request
    hooks=merge_hooks(request.hooks, self.hooks),
  File "/tmp/venv-core-selftest-agentwtd8oky_/lib/python3.4/site-packages/pip/_vendor/requests/models.py", line 287, in prepare
    self.prepare_url(url, params)
  File "/tmp/venv-core-selftest-agentwtd8oky_/lib/python3.4/site-packages/pip/_vendor/requests/models.py", line 338, in prepare_url
    "Perhaps you meant http://{0}?".format(url))
pip._vendor.requests.exceptions.MissingSchema: Invalid URL '/tmp/tmpgxqm1lttpip-cache-/cache': No schema supplied. Perhaps you meant http:///tmp/tmpgxqm1lttpip-cache-/cache?

Storing debug log for failure in /home/tarmac/.pip/pip.log
INFO: Executing: virtualenv -p python3 /tmp/venv-core-selftest-agentwtd8oky_
INFO: Executing: /tmp/venv-core-selftest-agentwtd8...

Read more...

Revision history for this message
Ubuntu CI Bot (uci-bot) wrote :

There are additional revisions which have not been approved in review. Please seek review and approval of these new revisions.

Revision history for this message
Paul Larson (pwlars) :
review: Needs Resubmitting
Revision history for this message
Para Siva (psivaa) wrote :

I see core-service.conf and SOLUTION_NAME=core-image-testing as potentially confusing but could notice some other services in the same solution are using this.

review: Approve
Revision history for this message
Paul Larson (pwlars) wrote :

Yeah, that's just a placeholder for now. A lot of that is going to need to change.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches