Skip to content
This repository has been archived by the owner on Jun 20, 2023. It is now read-only.

SubstanceD not correctly updated by buildout #2276

Closed
pallix opened this issue Apr 28, 2016 · 4 comments
Closed

SubstanceD not correctly updated by buildout #2276

pallix opened this issue Apr 28, 2016 · 4 comments
Labels

Comments

@pallix
Copy link
Collaborator

pallix commented Apr 28, 2016

Since #2192 is merged we have several times where SubstanceD failed to be updated to the correct new git commit:

  • in deployment when using salt
  • locally with buildout, when developing.
@pallix pallix added the backend label Apr 28, 2016
@joka
Copy link
Collaborator

joka commented Apr 29, 2016

Can you elaborate more what is causing trouble?

locally:

the mr.developer buildout extension takes care for git checkouts. The always-checkout
options controls if the checkouts are automaticall updated or not. The default is False.
We can change it to True or Force (override a dirty checkout).
The only downside is buildout takes more time to run.

deployment:

bin/ad_make_wheels creates a wheel from the local git checkout, but it does not handle checkouts of specific
commits or branches. The package name is the version number specified in setup.py, but not the commit or tag (like the output of git describe --tags)

ok, both is causing trouble i see.

@joka
Copy link
Collaborator

joka commented Apr 29, 2016

You can see the list of dependencies relying on git checkout here: src/adhocracy_core/sources.cfg

@pallix
Copy link
Collaborator Author

pallix commented May 2, 2016

@slomo what's your opinion on this?

@slomo
Copy link
Contributor

slomo commented May 9, 2016

In my opinion there is no specific deployment problem, because ad_make_wheels is supposed to run after buildout. So if buildout would work properly than we are fine.

This issue occurred not only on production but also on several developer machines where I manually fixed it. The repositories weren't dirty. In my opinion the recipe seems to be broken. We can try to find some one with an old installation where the issue still arises and debug a bit.

joka added a commit that referenced this issue May 25, 2016
All git repros checkedout by buildout ( src/adhocracy_core/sources.cfg)
are updated no if the tag/reversion number changes. Manual changes are
overriden.
joka added a commit that referenced this issue May 25, 2016
All git repros checkedout by buildout ( src/adhocracy_core/sources.cfg)
are updated no if the tag/reversion number changes. Manual changes are
overriden.
pallix pushed a commit that referenced this issue May 30, 2016
Fix #2276: force buildout to update git checkouts
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

3 participants