ui: improve docker reference parsing

[jgwhite]

Why the change?

Closes #2484 and #2495

What’s the plan?

• Add some failing tests
• Bring in a library to parse these references

What does it look like?

Before

🌝	🌚

After

🌝	🌚

CleanShot.2021-10-22.at.12.39.59.mp4

How do I test it?

1. Pull this branch down ui/docker-reference
2. Run the UI dev server
3. Visit localhost:4200
4. Tweak the image value in the resources factory
5. Verify these values show up correctly in the UI

Some examples to test with:

alpine
alpine@sha256:aaaaf56b44807c64d294e6c8059b479f35350b454492398225034174808d1726
registry:2
jpetazzo/pxe
jpetazzo/pxe:latest
quay.io/signalfuse/zookeeper
index.docker.io/library/ubuntu:latest
internal.mycorp.com:5000/revealjs
internal.mycorp.com:5000/revealjs:3.4.5-3
localhost:5000/mart/mass:latest
localhost:5000/mass

[jgwhite]

Looks like the prod build never completes!

[jgwhite]

OK, parse-docker-image-name is way too big. Back to the drawing board.

[gregone]

LGTM. Just a comment regarding the display of the full sha: in https://github.com/hashicorp/waypoint/blob/main/ui/app/components/git-commit.hbs#L3 I display the shortened version, but use the <CopyButton> to copy the full value. We could also update the truncate-commit helper to use an optional named length param with a default to 7 to make it applicable here.

[gregone]

LGTM. Just a comment regarding the display of the full sha: in https://github.com/hashicorp/waypoint/blob/main/ui/app/components/git-commit.hbs#L3 I display the shortened version, but use the <CopyButton> to copy the full value. We could also update the truncate-commit helper to use an optional named length param with a default to 7 to make it applicable here.

[jgwhite]

LGTM. Just a comment regarding the display of the full sha: in https://github.com/hashicorp/waypoint/blob/main/ui/app/components/git-commit.hbs#L3 I display the shortened version, but use the <CopyButton> to copy the full value. We could also update the truncate-commit helper to use an optional named length param with a default to 7 to make it applicable here.

Great idea. I’ll implement this before merging.