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

Add crowbar backup and restore logs to plugin #27

Merged
merged 1 commit into from
Nov 25, 2014

Conversation

nkrinner
Copy link
Contributor

Add crowbar batch export output to the plugin

@aspiers
Copy link
Contributor

aspiers commented Nov 10, 2014

+1, although we must not forget this assumes crowbar/barclamp-crowbar#1147 is merged, so it probably should not be merged first.

@aspiers
Copy link
Contributor

aspiers commented Nov 10, 2014

(If you want to play it safe, you could check for existence of /opt/dell/bin/crowbar_batch.)

@nkrinner
Copy link
Contributor Author

Blocked by crowbar/barclamp-crowbar#1147

@nkrinner nkrinner force-pushed the add-backup-and-batch-logs branch 2 times, most recently from 8586704 to 561fbf4 Compare November 11, 2014 15:39
@nkrinner
Copy link
Contributor Author

Dependency is now merged

@aspiers
Copy link
Contributor

aspiers commented Nov 12, 2014

crowbar/barclamp-crowbar#1152 has been submitted to backport the feature to stoney.

@nkrinner nkrinner force-pushed the add-backup-and-batch-logs branch 2 times, most recently from 6009237 to 569000a Compare November 14, 2014 15:13
Add crowbar batch export output to the plugin
Wipe passwords from tarball and delete crowbar batch export logfile
@nkrinner
Copy link
Contributor Author

Current version now also wiping passwords from crowbar patch export output. This should be fine now afaics.

@aspiers
Copy link
Contributor

aspiers commented Nov 17, 2014

+1 Looks great, thanks!

@dirkmueller
Copy link
Contributor

+1

dirkmueller added a commit that referenced this pull request Nov 25, 2014
Add crowbar backup and restore logs to plugin
@dirkmueller dirkmueller merged commit c3f0f36 into SUSE-Cloud:master Nov 25, 2014
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

Successfully merging this pull request may close these issues.

3 participants