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

ubic-admin --user-config option (and --no-user-config) #77

Open
tadam opened this issue Jun 8, 2015 · 1 comment
Open

ubic-admin --user-config option (and --no-user-config) #77

tadam opened this issue Jun 8, 2015 · 1 comment

Comments

@tadam
Copy link
Collaborator

tadam commented Jun 8, 2015

I'm using Ubic in C++ tests, so I cannot run ubic services just using Ubic.pm.
Typical workflow is:

  • Run ubic-admin setup to make all needed directories in a temp directory.
  • Put service files into tmp/ubic/service/
  • Run Ubic providing ENV vars.
  • ...
  • Stop Ubic using the same ENV vars.

So far everything is OK. But on the first step I cannot avoid reconfiguration of local ubic.cfg.
I'd like to have an option like --user-config and --no-user-config (similar to --crontab) to be able to turn off reconfiguration of ubic.cfg.

I don't send a patch as I don't know whether anything should be prompted for this option or not.

Another idea that I have is that instead of UBIC_DIR, UBIC_SERVICE_DIR, etc. you could have smth. like UBIC_CONFIG_FILE and all the other options would be placed in that file.

@berekuk
Copy link
Owner

berekuk commented Jul 23, 2015

This looks sane to me. I believe you won't have to prompt for anything (except for the config file name), since --batch-mode is already a thing.
UBIC_CONFIG_FILE is a good idea too.

I don't want to implement this myself, because even though the patch would be easy, I'm too lazy to write the proper tests, and I don't have any serious Ubic installations around to test it there. My perl skills are rusty...
I'll gladly merge a PR, though.

@berekuk berekuk closed this as completed Jul 23, 2015
@berekuk berekuk reopened this Jul 23, 2015
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