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

Error in 01-core-disk-space #46

Open
turb opened this issue Aug 14, 2013 · 6 comments
Open

Error in 01-core-disk-space #46

turb opened this issue Aug 14, 2013 · 6 comments

Comments

@turb
Copy link

turb commented Aug 14, 2013

Core disk space has a bug in some cases, maybe with some space formats.

Running scripts for stage pre-delivery
   pre-delivery/01-core-disk-space.sh
      Delivery will require 11.3086 MB on remote, 16.6601 GB available
bash: line 47: [[: 1.78886e+10: syntax error in expression (error token is ".78886e+10")
      Not enough disk space abvailable on remote
   Script returned with status 1
@turb
Copy link
Author

turb commented Aug 16, 2013

My mistake: git-deliver version was out to date.

@arnoo
Copy link
Owner

arnoo commented Jan 14, 2014

core-disk-space needs to be checked thoroughly anyway : I'm not sure it will work ok with most "df" versions and "awk" versions and variants.

@turb
Copy link
Author

turb commented Feb 26, 2014

Note I still have this error from time to time. It seems to be specific to a very precise pattern.

Workaround: change (even slightly) the free space.

@arnoo
Copy link
Owner

arnoo commented Feb 27, 2014

I looked at this a few days ago, and I have no idea how it could happen.

Are you positive you have the latest version of the core-disk-space script ? It could be you executed the --init with an old git-deliver version ?

If you do have the latest core-disk-space, any chance you could add a "set -x" at the top of it next time you encounter the problem and post the resulting trace here ?

@turb
Copy link
Author

turb commented Feb 27, 2014

It must be that, I forgot that updating git-deliver does not update previous presets.

I have to remind myself to overwrite my presets on each repo before next deliver.

Maybe a special option may do this, like --reset [presets]?

@arnoo
Copy link
Owner

arnoo commented Feb 27, 2014

Indeed, we could use an option to do that.

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

No branches or pull requests

2 participants