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

Ensure that branch has been updated to point off of master #143

Closed
lkysow opened this issue Sep 5, 2017 · 3 comments
Closed

Ensure that branch has been updated to point off of master #143

lkysow opened this issue Sep 5, 2017 · 3 comments

Comments

@lkysow
Copy link
Collaborator

lkysow commented Sep 5, 2017

Right now we don't check whether a branch has been updated to the latest master. We should just error out if it hasn't, otherwise the plan will be weird.

@grobie
Copy link

grobie commented Feb 15, 2018

That wouldn't work well for us. While we don't have a company-wide monorepo, we do have one for all infrastructure changes. There can be tens of unrelated commits in the same repository per hour.

A better approach would be to ensure that the commit of the latest applied state is in the history of the current commit. I'll explain that tomorrow in our call.

@lkysow
Copy link
Collaborator Author

lkysow commented Feb 15, 2018

Ahh interesting, I should have thought of that. Right now Atlantis just uses a file-based key-value store which is "okay" to lose because all it records are locks. It might be dangerous to rely on that store for looking up commit => state information.

At some point we'll move to an actual DB which might enable that better. I agree that that would be the most exact way to ensure the pull request is up to date.

@atlantisbot
Copy link

This issue was migrated to runatlantis/atlantis#35. Read about why here.

@hootsuite hootsuite locked and limited conversation to collaborators Mar 6, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

4 participants