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

feat(api/deploy): Default to redeploy if to_revision is not set #86

Merged
merged 2 commits into from Oct 22, 2015

Conversation

kaustavha
Copy link
Contributor

Allow automatic deploys without specifying to revision by making it default to current revision of that stack thats been deployed.

I noticed that in staging there's a specific revision that's been deployed over and over again, this should make that process easier.

Note: This PR does not make dreadnot auto-deploy to the latest git revision, only the git revision that's already been deployed and who's sha1 hash can be retrieved by running stack.getSummary.

…ion by making it default to current revision of that stack thats been deployed
@@ -77,34 +77,30 @@ exports.getAPIHandlers = function(dreadnot, authdb) {
// stack locked errors, redirect to the region view, otherwise a full error
// view.
function deploy(req, res) {
var stackName = req.params.stack,
var stackName = req.params.stack,
Copy link
Contributor

Choose a reason for hiding this comment

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

looks like your formatting is out of whack. var is indented 6 spaces..

@mburns mburns changed the title feat(api/deploy): Allow automatic deploys without specifying to revis… feat(api/deploy): Default to redeploy if to_revision is not set Oct 21, 2015
@mburns
Copy link
Contributor

mburns commented Oct 22, 2015

Needs to be rebased/linted, but then I'm 👍

@mburns
Copy link
Contributor

mburns commented Oct 22, 2015

fixed. 👍

mburns added a commit that referenced this pull request Oct 22, 2015
feat(api/deploy): Default to redeploy if to_revision is not set
@mburns mburns merged commit f69933d into master Oct 22, 2015
@jguice
Copy link
Contributor

jguice commented Oct 27, 2015

/opt/dn-bundle-4245263c548c2c7e052bce48ec63bced1db11011/lib/web/handlers/api.js:123
    getWarning: getWarning,
                ^
ReferenceError: getWarning is not defined
    at Object.exports.getAPIHandlers (/opt/dn-bundle-4245263c548c2c7e052bce48ec63bced1db11011/lib/web/handlers/api.js:123:17)
    at Object.exports.registerAPI (/opt/dn-bundle-4245263c548c2c7e052bce48ec63bced1db11011/lib/web/app.js:109:34)
    at Object.exports.run (/opt/dn-bundle-4245263c548c2c7e052bce48ec63bced1db11011/lib/web/app.js:167:27)
    at /opt/dn-bundle-4245263c548c2c7e052bce48ec63bced1db11011/lib/entry.js:57:12
    at /opt/dn-bundle-4245263c548c2c7e052bce48ec63bced1db11011/node_modules/async/lib/async.js:721:13
    at /opt/dn-bundle-4245263c548c2c7e052bce48ec63bced1db11011/node_modules/async/lib/async.js:52:16
    at /opt/dn-bundle-4245263c548c2c7e052bce48ec63bced1db11011/node_modules/async/lib/async.js:269:32
    at /opt/dn-bundle-4245263c548c2c7e052bce48ec63bced1db11011/node_modules/async/lib/async.js:44:16
    at /opt/dn-bundle-4245263c548c2c7e052bce48ec63bced1db11011/node_modules/async/lib/async.js:718:17
    at /opt/dn-bundle-4245263c548c2c7e052bce48ec63bced1db11011/node_modules/async/lib/async.js:167:37

Might have to revert this unless someone wants to pop in with a fix pretty quick ;)

@mburns mburns deleted the kaustavha/feat/auto-deploy branch August 1, 2018 23:27
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.

None yet

3 participants