loggerhead

Merge lp://staging/~jelmer/loggerhead/docker into lp://staging/loggerhead

  1. docker
  2. Merge into trunk-rich
Proposed by Jelmer Vernooij
Status: Merged
Approved by: Colin Watson
Approved revision: 514
Merge reported by: Otto Co-Pilot
Merged at revision: not available
Proposed branch: lp://staging/~jelmer/loggerhead/docker
Merge into: lp://staging/loggerhead
Diff against target: 17 lines (+13/-0)
1 file modified
Dockerfile (+13/-0)
To merge this branch: bzr merge lp://staging/~jelmer/loggerhead/docker
Reviewer Review Type Date Requested Status
Colin Watson (community) Approve
Review via email: mp+403635@code.staging.launchpad.net

Commit message

Add a basic Dockerfile for loggerhead.

Description of the change

Add a basic docker file for loggerhead.

To post a comment you must log in.
Revision history for this message
Colin Watson (cjwatson) wrote :

Thanks!

review: Approve
Revision history for this message
Otto Co-Pilot (otto-copilot) wrote :

A commit message must be set
https://jenkins.ols.canonical.com/online-services/job/loggerhead/10/

Revision history for this message
Otto Co-Pilot (otto-copilot) wrote :

Running landing tests failed
https://jenkins.ols.canonical.com/online-services/job/loggerhead/11/

Revision history for this message
Jelmer Vernooij (jelmer) wrote :

On Fri, Jun 11, 2021 at 03:45:25PM -0000, Otto Co-Pilot wrote:
> Running landing tests failed
> https://jenkins.ols.canonical.com/online-services/job/loggerhead/11/

Colin, happy to help debug this if necessary. I can't actually view
the logs though, so unsure if this is a problem with my merge proposal
or not.

--
Jelmer Vernooij <email address hidden>
PGP Key: https://www.jelmer.uk/D729A457.asc

Revision history for this message
Colin Watson (cjwatson) wrote :

Yep, sorry about that! The relevant bit of the failure is:

15:43:38 O: Collecting breezy (from -r requirements.txt (line 6))
15:43:38 O: Downloading https://files.pythonhosted.org/packages/09/71/b75b6f8872516a3fb5fa20f98f32097ac126a847cb90c5c9189fac5e47f9/breezy-3.2.0.tar.gz (9.4MB)
15:43:38 O: Complete output from command python setup.py egg_info:
15:43:38 O: [ERROR] Not a supported Python version. Need 3.5+

... so I guess that loggerhead needs to pin a version of breezy that supports Python 2 when it is running on Python 2 itself.

(It's a shame that breezy doesn't declare `python_requires` so that pip could figure this out for itself.)

Revision history for this message
Jelmer Vernooij (jelmer) wrote :

On Fri, Jun 11, 2021 at 04:36:57PM -0000, Colin Watson wrote:
> Yep, sorry about that! The relevant bit of the failure is:
>
> 15:43:38 O: Collecting breezy (from -r requirements.txt (line 6))
> 15:43:38 O: Downloading https://files.pythonhosted.org/packages/09/71/b75b6f8872516a3fb5fa20f98f32097ac126a847cb90c5c9189fac5e47f9/breezy-3.2.0.tar.gz (9.4MB)
> 15:43:38 O: Complete output from command python setup.py egg_info:
> 15:43:38 O: [ERROR] Not a supported Python version. Need 3.5+
>
> ... so I guess that loggerhead needs to pin a version of breezy that supports Python 2 when it is running on Python 2 itself.
 https://code.launchpad.net/~jelmer/loggerhead/pin-breezy should do this

> (It's a shame that breezy doesn't declare `python_requires` so that pip could figure this out for itself.)
Ah, that's a good point. I've submitted a merge proposal to set that.

Cheers,

Jelmer

--
Jelmer Vernooij <email address hidden>
PGP Key: https://www.jelmer.uk/D729A457.asc

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