Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(docker,titus): re-align digest field on image + tag selector #7526

Merged

Conversation

erikmunson
Copy link
Member

there were some odd hijinks in the docker image selector that seemed to be for compatibility with old bootstrap layouts. AFAICT that stuff no longer applies / is useful and has been causing some cool field alignment issues. Yanked it out and went through the usage of DockerImageAndTagSelector, seems to have not broken anything.

Before:
digest-before

After:
digest-after

Copy link
Contributor

@louisjimenez louisjimenez left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@christopherthielen christopherthielen merged commit c7feedd into spinnaker:master Oct 15, 2019
@erikmunson erikmunson deleted the fix-misaligned-digest-field branch October 15, 2019 22:00
christopherthielen added a commit that referenced this pull request Oct 15, 2019
c7feedd fix(docker,titus): re-align digest field on image + tag selector (#7526)
christopherthielen added a commit that referenced this pull request Oct 15, 2019
c7feedd fix(docker,titus): re-align digest field on image + tag selector (#7526)
christopherthielen added a commit to christopherthielen/deck that referenced this pull request Oct 15, 2019
c7feedd fix(docker,titus): re-align digest field on image + tag selector (spinnaker#7526)
christopherthielen added a commit to christopherthielen/deck that referenced this pull request Oct 15, 2019
c7feedd fix(docker,titus): re-align digest field on image + tag selector (spinnaker#7526)
yunzhangit pushed a commit to yunzhangit/deck that referenced this pull request Mar 28, 2021
c7feedd fix(docker,titus): re-align digest field on image + tag selector (spinnaker#7526)
yunzhangit pushed a commit to yunzhangit/deck that referenced this pull request Mar 28, 2021
c7feedd fix(docker,titus): re-align digest field on image + tag selector (spinnaker#7526)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants