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

Openshift VCR-recording script, touch up minishift instructions. #278

Merged
merged 5 commits into from
Feb 5, 2018

Conversation

cben
Copy link
Contributor

@cben cben commented Dec 26, 2017

cben added 3 commits December 26, 2017 15:13
- Use fixed dir also used by `minishift addons install --defaults`
- Use gist URL that works both for git clone and browser.
@cben
Copy link
Contributor Author

cben commented Dec 26, 2017

@enoodle @moolitayer please review

@cben
Copy link
Contributor Author

cben commented Jan 9, 2018

@zeari please review

$ minishift addons enable manageiq
```

3. Start minishift:
2. Start minishift:

```console
$ minishift start --vm-driver virtualbox --openshift-version "v1.5.0-rc.0"
Copy link

Choose a reason for hiding this comment

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

Is this the correct openshift version?

Copy link
Contributor Author

@cben cben Jan 11, 2018

Choose a reason for hiding this comment

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

updated to v3.6.1 which is what I used to record currently.
doesn't matter much, there is a link couple lines below to all valid docker tags.

@cben
Copy link
Contributor Author

cben commented Jan 14, 2018

@agrare @blomquisg please review/merge (not sure who are maintainers in this repo)

@agrare
Copy link
Member

agrare commented Jan 29, 2018

/cc @Fryguy

@cben
Copy link
Contributor Author

cben commented Jan 29, 2018

Updated per ManageIQ/manageiq-providers-openshift#83.

@chessbyte @Fryguy Who can merge PRs in this repo?

@himdel himdel merged commit 6684758 into ManageIQ:master Feb 5, 2018
@himdel himdel self-assigned this Feb 5, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants