search

integr8ly / tutorial-web-app

Solution Explorer provides the front door into the Integreatly initiative. It hosts the various Solution Patterns, as well as providing a dashboard of installed applications/products/services.
Apache License 2.0
36 stars 54 forks source link

INTLY-3511: Use shared Fuse for walkthroughs #521

Closed JameelB closed 4 years ago

JameelB commented 4 years ago

What

Use the shared fuse provisioned by the integreatly operator for walkthrough 1A instead of provisioning Fuse in the user namespace.

Why

By doing this, we won't need to add the pull secret to the walkthrough namespaces and no extra permissions are needed to be provided to the dev user for creating subscriptions.

Verification Steps

  1. Run the webapp and point it to an OpenShift 4 cluster.
    • Pass the following env vars when running yarn start:dev
    • OPENSHIFT_OAUTHCLIENT_ID, OPENSHIFT_OAUTH_HOST, OPENSHIFT_API, OPENSHIFT_HOST, SSO_ROUTE, OPENSHIFT_VERSION='4', INSTALLED_SERVICES
  2. Go to Walkthrough 1.
  3. Click on Get Started
    • Verify: Walkthrough 1A starts successfully.
  4. Go through Walkthrough 1A.
    • Verify: All the links to Fuse should be directed to the shared Fuse provisioned by the Integreatly operator.

Checklist:

Additional Notes

Related JIRA - https://issues.jboss.org/browse/INTLY-3511

JameelB commented 4 years ago

@philbrookes This should also work for walkthrough 2 but can't be tested atm as there's no support for walkthrough 2 yet.

philbrookes commented 4 years ago

Does this also need to be tested in Openshift 3? Or is Openshift 3 unaffected by these changes?

We verified these changes on OSD4 at the F2F right?

JameelB commented 4 years ago

@philbrookes There's 2 separate functions for preparing walkthroughs for openshift 3 and 4 at the moment so this should only affect openshift 4 :+1:

JameelB commented 4 years ago

@philbrookes Will hold off merge on this until I've synced up with @pwright and @aidenkeating on how this affects the walkthrough documentation.

JameelB commented 4 years ago

@pwright @aidenkeating Can you give this a look and make sure that you're ok with the changes made in this PR?