search

cisco / elsy

An opinionated, multi-language, build tool based on Docker and Docker Compose
Apache License 2.0
79 stars 22 forks source link

added upgrade #66

Closed joeygibson closed 7 years ago

joeygibson commented 7 years ago

This PR does a few things, none of which were asked for, but I think needed doing.

  1. When we use lc internally at Cisco, we have our own upgrade process. People outside Cisco don't, so I added lc system upgrade that will pull the latest release from Github and install it.
  2. I changed the imports for logrus from github.com/Sirupsen/logrus to github.com/sirupsen/logrus. Glide and Dep, one of which we may be using instead of Govendor at some point, but puke on that. I've also seen Govendor get into a weird state because of the capital 'S'.
  3. We misspelled snapshot in dev-env/lifecycle/package.
  4. I pegged the versions of turnip and turnip_formatter, because what we got without specifying a version doesn't actually work.

I had to do some really hinky things to test the various upgrade paths...

paulcichonski commented 7 years ago

Might not get to do a deep review until later tonight. Side note, we did have an upgrade command at one point, but removed it here 6333e0d (your reasons for adding it back make sense).