Page MenuHomeDevCentral

Add DockerHub support
ClosedPublic

Authored by dereckson on Sep 3 2016, 01:57.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Nov 18, 20:36
Unknown Object (File)
Thu, Nov 14, 17:40
Unknown Object (File)
Tue, Nov 12, 20:25
Unknown Object (File)
Mon, Nov 11, 02:08
Unknown Object (File)
Sat, Nov 9, 17:25
Unknown Object (File)
Sat, Nov 9, 16:48
Unknown Object (File)
Sat, Nov 9, 13:48
Unknown Object (File)
Wed, Nov 6, 23:37
Subscribers

Details

Summary

This change adds a service provider and a DockerHub facade to
be able to trigger a build for an image hosted on Docker Hub.

This will allow the notifications center to trigger new builds
when some event occurs, for example when app code is updated.

Ref T959.

Test Plan

Unit tests.

Tested in real conditions with:

$ php artisan tinker
Psy Shell v0.7.2 (PHP 5.6.16-1+deb.sury.org~trusty+1 — cli) by Justin Hileman
>>> $factory = App::make('dockerhub');
=> Keruald\DockerHub\Build\TriggerBuildFactory {#668}
>>> $factory->build('nasqueron/aphlict');
=> null
>>> $factory->build('nasqueron/aphlictx');
InvalidArgumentException with message 'No token found for image nasqueron/aphlictx.'
>>> DockerHub::build('nasqueron/auth-grove');
=> null

Diff Detail

Repository
rNOTIF Notifications center
Lint
Lint Passed
Unit
Tests Passed
Branch
service-provider-dockerhub (branched from master)

Event Timeline

dereckson retitled this revision from to Add DockerHub support.
dereckson updated this object.
dereckson edited the test plan for this revision. (Show Details)
dereckson added a reviewer: dereckson.
dereckson added inline comments.
storage/app/.gitignore
3

\n

tests/data/DockerHubTokens.json
4

\n

This revision is now accepted and ready to land.Sep 3 2016, 02:01
dereckson edited edge metadata.
dereckson marked 2 inline comments as done.

Ready to merge.

This revision was automatically updated to reflect the committed changes.