Skip to content
This repository was archived by the owner on Nov 16, 2020. It is now read-only.

CLI: dispatch create seed-images #507

Merged
merged 2 commits into from
Jun 13, 2018
Merged

Conversation

imikushin
Copy link
Contributor

When used without any arguments, creates base-images and images from
images.yaml at the same revision as the CLI build. When used with
-O file.yaml flag, writes the YAML to the specified file (nothing gets
created).

Also:

  • Bump base-image versions
  • Move images.yaml to the repo root

When used without any arguments, creates base-images and images from
images.yaml at the same revision as the CLI build. When used with
`-O file.yaml` flag, writes the YAML to the specified file (nothing gets
created).

Also:
- Bump base-image versions
- Move images.yaml to the repo root
kars7e
kars7e previously approved these changes Jun 12, 2018
Copy link
Contributor

@kars7e kars7e left a comment

Choose a reason for hiding this comment

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

LGTM

Makefile Outdated
GO_LDFLAGS += -X $(VERSION_PACKAGE).buildDate=$(shell date +'%Y-%m-%dT%H:%M:%SZ')
GO_LDFLAGS += -X $(VERSION_PACKAGE).commit=$(shell git rev-parse HEAD)
GO_LDFLAGS +="

CLI_LDFLAGS := -X $(CLICMD_PACKAGE).imagesB64=$(shell cat images.yaml | gzip | base64)
Copy link
Contributor

Choose a reason for hiding this comment

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

NIT: could be gzip -c images.yaml | base64, one cat's life saved ;)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

No! This would kill the cat! 🙀

@neosab neosab merged commit 07693ee into vmware-archive:master Jun 13, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants