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

Getting an error when simulate #113

Closed
planetahuevo opened this Issue Aug 29, 2017 · 8 comments

Comments

Projects
None yet
2 participants
@planetahuevo

planetahuevo commented Aug 29, 2017

I am getting this error on a simulate.
Testing right now the backup to see if I have the same issue.

Exception 'RuntimeException' with message 'SplFileInfo::getMTime(): stat failed for /****/filesbackup.xxx' in phar:///usr/local/bin/phpbu/Backup/File.php:94

@planetahuevo

This comment has been minimized.

Show comment
Hide comment
@planetahuevo

planetahuevo Aug 29, 2017

Same backup without simulate works fine.

planetahuevo commented Aug 29, 2017

Same backup without simulate works fine.

@sebastianfeldmann

This comment has been minimized.

Show comment
Hide comment
@sebastianfeldmann

sebastianfeldmann Aug 30, 2017

Owner

What kind of backup is causing the problem?

<backup type="???">

Can you please provide the backup configuration?

<backup>
...
</backup>
Owner

sebastianfeldmann commented Aug 30, 2017

What kind of backup is causing the problem?

<backup type="???">

Can you please provide the backup configuration?

<backup>
...
</backup>
@sebastianfeldmann

This comment has been minimized.

Show comment
Hide comment
@sebastianfeldmann

sebastianfeldmann Aug 30, 2017

Owner

Would be great if you could remove all extra configuration check, crypt, sync and cleanup to narrow down which simulation causes the problem

Owner

sebastianfeldmann commented Aug 30, 2017

Would be great if you could remove all extra configuration check, crypt, sync and cleanup to narrow down which simulation causes the problem

@planetahuevo

This comment has been minimized.

Show comment
Hide comment
@planetahuevo

planetahuevo Sep 1, 2017

I was able to debug the problem to the cleanup code.
I have this:

<!-- cleanup capacity: clean all -->
	<cleanup type="capacity">
	  <option name="size" value="0B"/>
	  <option name="deleteTarget" value="true"/>
	</cleanup>

Not sure where did I get the value field, maybe docs. I am sure I read about that somewhere.

This code works fine without simulate option and it worked fine with simulate on 4.X version.

planetahuevo commented Sep 1, 2017

I was able to debug the problem to the cleanup code.
I have this:

<!-- cleanup capacity: clean all -->
	<cleanup type="capacity">
	  <option name="size" value="0B"/>
	  <option name="deleteTarget" value="true"/>
	</cleanup>

Not sure where did I get the value field, maybe docs. I am sure I read about that somewhere.

This code works fine without simulate option and it worked fine with simulate on 4.X version.

@sebastianfeldmann

This comment has been minimized.

Show comment
Hide comment
@sebastianfeldmann

sebastianfeldmann Sep 1, 2017

Owner

Thanks for taking the time to debug this
I will have a look at the Cleanup simulation code and hopefully release a fix soon

Owner

sebastianfeldmann commented Sep 1, 2017

Thanks for taking the time to debug this
I will have a look at the Cleanup simulation code and hopefully release a fix soon

@planetahuevo

This comment has been minimized.

Show comment
Hide comment
@planetahuevo

planetahuevo Sep 1, 2017

Is the least I can do. Happy to help on what I can. :)

planetahuevo commented Sep 1, 2017

Is the least I can do. Happy to help on what I can. :)

@planetahuevo

This comment has been minimized.

Show comment
Hide comment
@planetahuevo

planetahuevo Sep 1, 2017

One more thing, I only have that cleanup option with a tar backup.
The quantity I use for the db backups works fine.

planetahuevo commented Sep 1, 2017

One more thing, I only have that cleanup option with a tar backup.
The quantity I use for the db backups works fine.

@sebastianfeldmann sebastianfeldmann self-assigned this Sep 18, 2017

sebastianfeldmann added a commit that referenced this issue Sep 19, 2017

Fix 'deleteTarget' simulation problem - issue #113
There was an error if the 'deleteTarget' option was activated
during simulation. The target is getting added to the files
to delete list and the Cleaner was trying to get the MTime
and Size from a file that doesn't exist.
To fix this we deactivate the option during simulation and
add some hints to the output, that the target will be deleted
as well.
@sebastianfeldmann

This comment has been minimized.

Show comment
Hide comment
@sebastianfeldmann

sebastianfeldmann Sep 19, 2017

Owner

I just released version 5.0.4 with a fix for this issue ;)

Owner

sebastianfeldmann commented Sep 19, 2017

I just released version 5.0.4 with a fix for this issue ;)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment