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

UBI to OpenShift 4 (beta) #8

Open
wants to merge 16 commits into
base: master
Choose a base branch
from
Open

UBI to OpenShift 4 (beta) #8

wants to merge 16 commits into from

Conversation

Grant-Steinfeld
Copy link
Collaborator

Documentation of UBI to OpenShift 4 (beta) experience

@MaxShapiro32 let's review and troubleshoot - all worked except the developer web console perspective to create an app using the container image, Event Log errors in README - invalid image name

GrantSteinfeld and others added 16 commits February 9, 2020 18:38
* root gitignore for python

* adding python-starter code
* root gitignore for python

* adding python-starter code

* Currency Exchange coding starts

* reads csv file but still hard codes the getCurrencyNameAndCode()

* adding country currency code meta

* basic code now works against tests

* tests for csv reading now pass

* urllib now works

* test works against live api, mock data fixture monkeypatch not yet resolved

* api call works tests pass

* main is now flask api entrypoint

* flask-restplus now works as expected
in order for `flask-restplus` requires this as to work properly with `Werkzeug` pinned to earlier version ~= 0.16.0
* hard coded value

* get all currency rates

* Swagger updated to all endpoints
* trying alpine docker first

* alpine base image works now

* readme formatting

* essentials only

* added instructions for UBI

* image to get docker hub user credentials

* new image for red-green-refactor

* cleaned up
@Grant-Steinfeld Grant-Steinfeld self-assigned this Feb 20, 2020
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.

1 participant