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

Show image tag in node details #3173

Merged
merged 3 commits into from
May 17, 2018
Merged

Show image tag in node details #3173

merged 3 commits into from
May 17, 2018

Conversation

fbarl
Copy link
Contributor

@fbarl fbarl commented May 15, 2018

Resolves #3161.

image

image

@fbarl fbarl self-assigned this May 15, 2018
@fbarl fbarl force-pushed the 3161-show-image-tag branch 3 times, most recently from 941fec9 to ad6d996 Compare May 15, 2018 15:06
@fbarl fbarl changed the title [WIP] Show image tag in node details Show image tag in node details May 15, 2018
@fbarl fbarl requested a review from lilic May 15, 2018 15:06
Copy link
Contributor

@lilic lilic left a comment

Choose a reason for hiding this comment

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

Few nits and suggestions. :)

// ImageNameWithoutVersion splits the image name apart, returning the name
// without the version, if possible
func ImageNameWithoutVersion(name string) string {
// ImageNameParts returns parts of the full image name (image name, image tag).

This comment was marked as abuse.

This comment was marked as abuse.


// ImageNameWithoutVersion splits the image name apart, returning the name
// without the version, if possible
func ImageNameWithoutVersion(name string) string {

This comment was marked as abuse.

@fbarl
Copy link
Contributor Author

fbarl commented May 16, 2018

@lilic PTAL.

@fbarl fbarl force-pushed the 3161-show-image-tag branch from 8e846b7 to bfb20a8 Compare May 17, 2018 09:44
@fbarl fbarl requested a review from dlespiau May 17, 2018 10:17
ImageName: "Name",
// Prepend spaces as a hack to keep at the top when sorted.
ImageID: " ID",
ImageName: " Name",

This comment was marked as abuse.

This comment was marked as abuse.

@fbarl fbarl merged commit 2e8c996 into master May 17, 2018
@fbarl fbarl deleted the 3161-show-image-tag branch May 17, 2018 15:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants