Git Product home page Git Product logo

Comments (12)

bacongobbler avatar bacongobbler commented on August 24, 2024

@helgi do you happen to have a checklist handy of all the resources we need to imitate in the controller for the router? I know we've tackled domains, but I guess app config like IP whitelisting and certs still need to be done.

from controller.

krancour avatar krancour commented on August 24, 2024

@bacongobbler see annotation reference in https://github.com/deis/router#annotations

The four app-level things that can be tweaked at present are domains, connect timeout, read/write timeout, and whitelists.

from controller.

helgi avatar helgi commented on August 24, 2024

Certificates were the only thing on my list so far - We may be able to pull Config over and use that as a way to cover for IP whitelisting and such. Shouldn't be hard if we come up with a convention (emulated deis.io/routerConfig)

I know that is a bigger scope than router but might be worth it

from controller.

krancour avatar krancour commented on August 24, 2024

We don't need to concern ourselves with certs for alpha.

from controller.

helgi avatar helgi commented on August 24, 2024

Agreed but I may start work on that so the workflow side of it is done before we touch the router itself

from controller.

helgi avatar helgi commented on August 24, 2024

Removing showstopper label since this is a broader ticket

from controller.

krancour avatar krancour commented on August 24, 2024

If this is broadening to that level, the alpha milestone on the ticket is wrong. I'm thinking we need separate issues for achieving the minimum functionality required for alpha and then anything else that's for beta and beyond.

fwiw... I just realized the controller does not need to add any support for setting timeouts, even though the router supports it. v1.x did not have this feature and the only reason I added support for this to the router itself was that it allowed me to treat the controller as "just another app." (Controller itself needed those values tweaked-- see the chart-- but does not need to manage them on other apps in order to achieve feature parity.)

https://github.com/deis/charts/blob/master/deis/manifests/deis-workflow-service.yaml#L13-L14

from controller.

krancour avatar krancour commented on August 24, 2024

So if workflow can handle whitelists, we're in decent shape.

from controller.

bacongobbler avatar bacongobbler commented on August 24, 2024

yeah I imagined this was a post-alpha milestone. Not something we need to tackle today if the basic workflow is good (deis open)

from controller.

technosophos avatar technosophos commented on August 24, 2024

This is on the beta list, but it's not clear to me what is left to do. Can someone update a status on it? And maybe assign it to themselves?

from controller.

helgi avatar helgi commented on August 24, 2024

The IP whitelist is not really done but we haven't really planned it either. Not have we figured out how we want to do configuration changes (if at all) that affect other deis components, ala deisctl.

Removing it from beta for now but we need to get to this and make decisions sooner or later

from controller.

helgi avatar helgi commented on August 24, 2024

Closing this since we got all the items above, last one was IP Whitelist, and we have a way we like (so far) to modify things like the router, etc

from controller.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.