Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@lildude
Copy link
Member

@lildude lildude commented Jan 28, 2015

Several people have been a little surprised to find their configuration isn't restored to the destination host as they know nothing about the -c option and I don't think many people query ghe-restore to find out the available options.

This PR documents this option.

@rtomayko
Copy link
Contributor

Yeah, the behavior here is a little weird. Settings are restored automatically if the instance has never been configured (migrate flow) but must be specified explicitly if restoring over an already configured instance. We'll need to incorporate that bit of nuance here.

@tedivm
Copy link

tedivm commented May 5, 2015

This would have been good to know. Any reason this little tidbit hasn't been merged in yet?

@rtomayko
Copy link
Contributor

rtomayko commented May 7, 2015

@tedvim Nope, just fell off my radar. I want to clarify the bit about new vs. already configured instances and then I'll get this merged down.

@xeago
Copy link
Contributor

xeago commented May 8, 2015

While we're at it, I'd like to make a feature request for overriding a single parameter when restoring: the hostname. The hostname is used to identify the instance within collectd - restoring a the configuration with github.example.com will mess with the monitoring setup. See the image attached where the hostname changed to something that is not our primary host at 10:31.

screen shot 2015-05-08 at 07 23 07

@rtomayko
Copy link
Contributor

Tweaked the README copy to clarify the difference between restoring to an already configured instance vs. an instance that's never been configured in b4b8bdb.

I also created #128 to capture @xeago's request for overriding configuration on restore.

Merging ...

rtomayko added a commit that referenced this pull request May 11, 2015
@rtomayko rtomayko merged commit 8bed83e into master May 11, 2015
@rubiojr rubiojr deleted the doc-settings-opt branch February 25, 2016 17:23
Copy link

@candacebutler8879 candacebutler8879 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok

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.

6 participants