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

added support for old bugged refs/tags #205

Merged
merged 3 commits into from
Jan 29, 2016
Merged

Conversation

mjksmith
Copy link
Contributor

Bugged == of the format 'paasta-paasta-cluster.instance'.

Also added an itest that makes sure generate deployments works properly with force_bounce and state tags.

@mjksmith mjksmith self-assigned this Jan 29, 2016
Given a test git repo is setup with commits
When we generate deployments.json for that service
Given a test git repo is setup with commits
When paasta stop is run against the repo
Copy link
Contributor

Choose a reason for hiding this comment

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

Thanks! This is good stuff.

@solarkennedy
Copy link
Contributor

fix + ship. I would like to get @EvanKrall's official opinion on this whole paasta-, paasta-paasta- nonsense.

If the itest paases, then wfm

@@ -57,6 +60,30 @@ def step_impl_given(context):
context.expected_commit = commit.id


@when(u'paasta stop is run against the repo')
Copy link
Contributor

Choose a reason for hiding this comment

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

It might be nice to parameterize this step so that you can run any paasta command against it.

mjksmith added a commit that referenced this pull request Jan 29, 2016
added support for old bugged refs/tags
@mjksmith mjksmith merged commit 9f78472 into master Jan 29, 2016
@mjksmith mjksmith deleted the fix_refs_tags_deployments branch January 29, 2016 00:55
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