New Bundle: docker-core

Bug #1413779 reported by Charles Butler
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Juju Charms Collection
Expired
Critical
Unassigned

Bug Description

Here's a bundle that wraps up the previous 2 submissions in bug:

1413775 and 1413776

I've got a pending meta item for the full documentation suite, however the README does attempt to describe the full service suite with deployment guide. There is a full stack integration test included in tests/ for evaluation.

The prior charms are expected to be promulgated in the charm test suite, and will cause false positives from bundletester when put under evaluation when testing this bundle - please keep the chicken and egg scenario in mind when reviewing.

I look forward to any feedback you might have

Revision history for this message
Review Queue (review-queue) wrote : Automated Test Results: New Bundle: docker-core

This items has failed automated testing! Results available here http://reports.vapour.ws/charm-tests/charm-bundle-test-10973-results

Revision history for this message
Review Queue (review-queue) wrote :

This items has failed automated testing! Results available here http://reports.vapour.ws/charm-tests/charm-bundle-test-10995-results

Revision history for this message
Whit Morriss (whitmo) wrote :

This charm will not pass on LXC at this time.

Changed in charms:
importance: Undecided → Critical
status: New → In Progress
Revision history for this message
Whit Morriss (whitmo) wrote :

Thanks for this bitchin' bundle Chuck!

Looks like there are some general testing issues beyond the fact that deploying docker on LXC is not something we expect to work.

I also would like to see some more info on how to exercise the resulting deployment (use docker, use flannel, etc). I've issued a PR describing how to set up a trivial nginx container that can be hit from the internet.

https://github.com/whitmo/docker-bundle/commit/0b373670cbf882c5e2baff6bafc2e24ba5d5d4a5

 I like your idea of extending this to serve a couple nodes of ttrss backed by a containerized postgres to demo how flannel provides value.

I also would like to see the docs link at the top of Readme (or woven into each section) rather than below the fold.

so -1 for now but plus uno soon! thanks again Chuck!

-w

Changed in charms:
status: In Progress → Fix Committed
Changed in charms:
status: Fix Committed → In Progress
Changed in charms:
status: In Progress → Incomplete
Revision history for this message
Launchpad Janitor (janitor) wrote :

[Expired for Juju Charms Collection because there has been no activity for 60 days.]

Changed in charms:
status: Incomplete → Expired
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.