Added option to define config.js for st2web

[moonrail]

Hello maintainers,

while testing and developing internally we've experienced the requirement, where we would have to access StackStorm in a subpath of a FQDN, e.g. sub.domain.tld/stackstorm.

For this configuration is required for Ingress and st2web.

This PR adds the possibility to define configuration for st2web.

Ingress is pretty straight forward and already supported, e.g.:

ingress:
    enabled: true
    annotations:
      kubernetes.io/ingress.class: nginx
      kubernetes.io/tls-acme: 'true'
      nginx.ingress.kubernetes.io/rewrite-target: /$1
    hosts: 
      - host: 'sub.domain.tld'
        paths:
          - path: '/stackstorm/(.*)'
            serviceName: st2web
            servicePort: 80
    tls:
      - secretName: 'tls-sub.domain.tld'
        hosts:
          - 'sub.domain.tld'

With this Ingress configuration, st2web is properly reachable.

By default st2web uses relative paths to its domain to access services like auth, api, and stream. So for our example, st2web would try to authenticate against sub.domain.tld/auth.

A solution is described here: https://github.com/StackStorm/st2web#connecting-to-st2-server

In short: Edit /opt/stackstorm/static/webui/config.js to include complete Endpoints for auth, api, and stream.

For our example config.js could be:

'use strict';

/* global angular */
angular.module('main')
  .constant('st2Config', {

    hosts: [
      {
        name: 'test',
        url: 'https://sub.domain.tld/stackstorm/api',
        auth: 'https://sub.domain.tld/stackstorm/auth',
        stream: 'https://sub.domain.tld/stackstorm/stream',
      }
    ]

  });

[arm4b]

Thanks for the contribution! That's a nice composed feature 👍

I just left one comment before we can merge it.

[arm4b]

Please also sync up your PR branch with the latest master, - that'll fix the helm-lint CI failures.

[moonrail]

Rebased :)

[arm4b]

Thanks @moonrail!
The only missing thing to merge it is checksum adjustment for the new configmap/deployment, see the review comment above ^^: #165 (comment)

[moonrail]

@armab
Sorry, I've been busy and overlooked your comment - Checksum should be generated as with other deployments as of now.

[arm4b]

You're definitely faster then I am with the other PRs :)

[arm4b]

Nice feature, thanks a lot for implementation! 👍