Skip to content

Conversation

@sdodson
Copy link
Member

@sdodson sdodson commented Feb 10, 2015

This is temporary. If we're looking for this to be a complete solution for OSE we'll also need to modify the image name.

@sdodson
Copy link
Member Author

sdodson commented Feb 10, 2015

@pweil- and @sosiouxme relevant to your interests

@sosiouxme
Copy link
Member

Need the same thing for the docker registry (so it can alert the master when it gets updated images).

https://github.com/openshift/origin/blob/master/hack/install-registry.sh

@pweil-
Copy link

pweil- commented Feb 10, 2015

I'm starting to think we should just convert this to use getopts. The next person that has another file/cert/whatever is going to run into the same trouble but hack/install-router -r ~/route.json -c ~/thecert.pem -i routerId -m https://masterurl:8443 seems like a decent solution.

What do you think @sosiouxme @thoraxe @sdodson?

@thoraxe
Copy link
Contributor

thoraxe commented Feb 10, 2015

It would be nice if you could specify files and other things on the cli, and have the script automatically ingest them and turn them into appropriate vars.

@sdodson
Copy link
Member Author

sdodson commented Feb 10, 2015

I was going to suggest that we convert pod.json into a template similar to how install-registry.sh and examples/sample-app/docker-registry-template.json do things. But I found out that template parameters are only evaluated for environment variables so this won't work for adjusting the image name from 'openshift/docker-registry' to 'openshift3_beta/docker-registry' so we're stuck with sed manipulation regardless.

We could just keep adding things to /etc/sysconfig/openshift and have scripts source that file if it exists and has the variable we want otherwise use script local defaults. This may be nice because it would promote re-use, i hope, perhaps not in practice.

@openshift-bot
Copy link
Contributor

Origin Action Required: Pull request cannot be automatically merged, please rebase your branch from latest HEAD and push again

@smarterclayton
Copy link
Contributor

#1053 removes the need for this pull. Router installation is a separate command, and hack/install-router is gone.

@detiber
Copy link
Contributor

detiber commented Feb 20, 2015

#1043 Is the correct PR for the router command

@sdodson sdodson closed this Feb 20, 2015
@sdodson sdodson deleted the install-router branch May 11, 2018 19:29
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.

7 participants