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 origin- prefix to ansible-service-broker image #119

Merged
merged 1 commit into from Aug 15, 2017

Conversation

djzager
Copy link
Contributor

@djzager djzager commented Aug 15, 2017

The work to create the origin-ansible-service-broker image has already
been done via Jenkins pipelines and automated builds on dockerhub. This
makes it so that automated environment setups get the correct image
(origin-ansible-service-broker).

The work to create the origin-ansible-service-broker image has already
been done via Jenkins pipelines and automated builds on dockerhub. This
makes it so that automated environment setups get the correct image
(`origin-ansible-service-broker`).
@djzager
Copy link
Contributor Author

djzager commented Aug 15, 2017

Tests

Running catasb

➜  linux git:(use-origin) ✗ ./run_setup_local.sh
 [WARNING]: Host file not found: /etc/ansible/hosts

 [WARNING]: provided hosts list is empty, only localhost is available


PLAY [localhost] **************************************************************************************************************************************************************************************************************************************************************

TASK [Gathering Facts] ********************************************************************************************************************************************************************************************************************************************************
ok: [localhost]

...

TASK [openshift_setup : Removing certain docker images if they exist so we are sure we are pulling latest] ********************************************************************************************************************************************************************
skipping: [localhost] => (item={u'tag': u'latest', u'img': u'docker.io/ansibleplaybookbundle/origin-ansible-service-broker'})

TASK [openshift_setup : Pulling all docker images we require] *****************************************************************************************************************************************************************************************************************
ok: [localhost] => (item={u'tag': u'latest', u'img': u'docker.io/ansibleplaybookbundle/origin-ansible-service-broker'})
ok: [localhost] => (item={u'tag': u'latest', u'img': u'quay.io/coreos/etcd'})
ok: [localhost] => (item={u'tag': u'v3.6.0-rc.0', u'img': u'docker.io/openshift/origin'})
ok: [localhost] => (item={u'tag': u'v3.6.0-rc.0', u'img': u'docker.io/openshift/origin-sti-builder'})
ok: [localhost] => (item={u'tag': u'v3.6.0-rc.0', u'img': u'docker.io/openshift/origin-deployer'})
ok: [localhost] => (item={u'tag': u'v3.6.0-rc.0', u'img': u'docker.io/openshift/origin-docker-registry'})
ok: [localhost] => (item={u'tag': u'v3.6.0-rc.0', u'img': u'docker.io/openshift/origin-pod'})
ok: [localhost] => (item={u'tag': u'v3.6.0-rc.0', u'img': u'docker.io/openshift/origin-haproxy-router'})
ok: [localhost] => (item={u'tag': u'v3.6.0-rc.0', u'img': u'docker.io/openshift/origin-service-catalog'})

...

TASK [debug] ******************************************************************************************************************************************************************************************************************************************************************
ok: [localhost] => {
    "msg": [
        "Hostname:                  172.17.0.1",
        "Next steps:",
        "Visit https://172.17.0.1:8443 for the web console",
        "OR",
        "For CLI access:",
        "oc login --insecure-skip-tls-verify 172.17.0.1:8443 -u admin -p admin",
        ""
    ]
}

PLAY RECAP ********************************************************************************************************************************************************************************************************************************************************************
localhost                  : ok=61   changed=25   unreachable=0    failed=0

➜  linux git:(use-origin) ✗ oc get pods -n ansible-service-broker asb-1-b68ph -o yaml
apiVersion: v1
kind: Pod
metadata:
...
spec:
  containers:
  - env:
    - name: BROKER_CONFIG
      value: /etc/ansible-service-broker/config.yaml
    - name: INSECURE
      value: "false"
    image: docker.io/ansibleplaybookbundle/origin-ansible-service-broker:latest
    imagePullPolicy: IfNotPresent
    name: asb

@djzager djzager changed the title Add origin- prefix to ansible-service-broker image [WIP] Add origin- prefix to ansible-service-broker image Aug 15, 2017
@djzager djzager changed the title [WIP] Add origin- prefix to ansible-service-broker image Add origin- prefix to ansible-service-broker image Aug 15, 2017
@djzager
Copy link
Contributor Author

djzager commented Aug 15, 2017

I had marked this WIP because I thought it may be necessary for the broker project to be updated first (namely the broker template). This isn't the case though since the template is rendered with the image we want.

@jmontleon
Copy link
Contributor

ACK

@jmontleon jmontleon merged commit 688ea5a into fusor:master Aug 15, 2017
@djzager
Copy link
Contributor Author

djzager commented Aug 16, 2017

shawn-hurley pushed a commit to shawn-hurley/catasb that referenced this pull request Sep 12, 2017
The work to create the origin-ansible-service-broker image has already
been done via Jenkins pipelines and automated builds on dockerhub. This
makes it so that automated environment setups get the correct image
(`origin-ansible-service-broker`).
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.

None yet

2 participants