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

plugin failing on one test deploy #1

Open
lepinsk opened this issue Mar 26, 2015 · 2 comments
Open

plugin failing on one test deploy #1

lepinsk opened this issue Mar 26, 2015 · 2 comments
Assignees
Labels

Comments

@lepinsk
Copy link
Owner

lepinsk commented Mar 26, 2015

One repo we're testing this on (on Heroku's cedar-10) seems to have a push_metadata.yml file that is considerably different from what we're seeing everywhere else in our testing.

Initially I thought this was a cedar-10 vs cedar-14 issue, but I just created a test app on Heroku, switched the stack to cedar-10 and push_metadata.yml indeed has source_blob.version as we'd expected.

I wonder if this is somehow related to another configuration option on the app? I know that the path where the slug building occurs is different depending on whether the build process is kicked off by a git push to Heroku, or a hook triggered on a GitHub push (the latter including the git hash as part of the path, the former not). Could some other option (module caching?) be causing this? What else could it be?

The other thing that differs in this case (the failing case) is that the push_metadata.yml file contains a commit_hash key, where the value seems to be the previous commit to the current one. (Again, caching?)

@lepinsk lepinsk self-assigned this Mar 26, 2015
@lepinsk lepinsk added the bug label Mar 26, 2015
@lepinsk
Copy link
Owner Author

lepinsk commented Mar 26, 2015

Is it possibly related to this being built for a dual-dyno system? I wonder if the build system for that differs.

A tiny bit of info: https://devcenter.heroku.com/changelog-items/521

@lepinsk
Copy link
Owner Author

lepinsk commented Mar 26, 2015

Just tested with a dual-dyno setup. No luck.

:rage4:

lepinsk added a commit that referenced this issue Mar 26, 2015
* documenting the known issue in #1
* throwing an error in this case
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant