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

Ability to set "exposure" by process #1076

Open
iserko opened this issue May 3, 2017 · 3 comments
Open

Ability to set "exposure" by process #1076

iserko opened this issue May 3, 2017 · 3 comments

Comments

@iserko
Copy link
Contributor

iserko commented May 3, 2017

Currently "exposure" as it is phrased in the code is controlled by running:

  • emp domain-add <domain> which makes ELB/ALB internet-facing
  • emp domain-remove <domain> which makes ELB/ALB internal (or if you never use domain-add for an app

That setting is applied across the whole application and has no ability to set it per web process.

What I would like to see is to be able to control it via the Procfile using EMPIRE_X_EXPOSURE if it existed:

# internal
qint:
  command: ./bin/web
  ports:
    - "80:8080"
  environment:
    EMPIRE_X_LOAD_BALANCER_TYPE: "alb"
    EMPIRE_X_EXPOSURE: "internal"

# external
qext:
  command: ./bin/web
  ports:
    - "80:8080"
  environment:
    EMPIRE_X_LOAD_BALANCER_TYPE: "alb"
    EMPIRE_X_EXPOSURE: "internet-facing"  
@ejholmes
Copy link
Contributor

ejholmes commented May 3, 2017

Yeah, I've hit this before as well. The domain-* stuff was always meant to be pulled out before a 1.0 release, since it's mostly cruft leftover from the first version. Per-process exposure in the Procfile like this makes a lot more sense, the only complication is with pulling it out is backwards compat.

@iserko
Copy link
Contributor Author

iserko commented May 3, 2017

@ejholmes well I would say that if you have EMPIRE_X_EXPOSURE set ... that would take priority ... then you just deprecate domains-add ... the only reason why we ever even use domains-add is to make the ELB/ALB public.

In what way is backwards compatibility an issue?

@ejholmes
Copy link
Contributor

ejholmes commented May 3, 2017

Not that it's an issue, just needs to be considered. I think ^ is the right way to approach it.

iserko added a commit to iserko/empire that referenced this issue May 5, 2017
…ation

Introduces EMPIRE_X_EXPOSURE which needs to be either `internal` or `internet-facing` (kept AWS terminology here).

If that environment variable is not set, we fallback to the default selection via application (where you use `domain-add`).

Fixes remind101#1076
iserko added a commit to iserko/empire that referenced this issue Jun 13, 2017
…ation

Introduces EMPIRE_X_EXPOSURE which needs to be either `internal` or `internet-facing` (kept AWS terminology here).

If that environment variable is not set, we fallback to the default selection via application (where you use `domain-add`).

Fixes remind101#1076
iserko added a commit to iserko/empire that referenced this issue Jun 13, 2017
…ation

Introduces EMPIRE_X_EXPOSURE which needs to be either `internal` or `internet-facing` (kept AWS terminology here).

If that environment variable is not set, we fallback to the default selection via application (where you use `domain-add`).

Fixes remind101#1076
iserko added a commit to iserko/empire that referenced this issue Jun 13, 2017
…ation

Introduces EMPIRE_X_EXPOSURE which needs to be either `internal` or `internet-facing` (kept AWS terminology here).

If that environment variable is not set, we fallback to the default selection via application (where you use `domain-add`).

Fixes remind101#1076
iserko added a commit to iserko/empire that referenced this issue Jun 13, 2017
…ation

Introduces EMPIRE_X_EXPOSURE which needs to be either `internal` or `internet-facing` (kept AWS terminology here).

If that environment variable is not set, we fallback to the default selection via application (where you use `domain-add`).

Fixes remind101#1076
iserko added a commit to iserko/empire that referenced this issue Jun 13, 2017
…ation

Introduces EMPIRE_X_EXPOSURE which needs to be either `internal` or `internet-facing` (kept AWS terminology here).

If that environment variable is not set, we fallback to the default selection via application (where you use `domain-add`).

Fixes remind101#1076
iserko added a commit to iserko/empire that referenced this issue Jun 13, 2017
…ation

Introduces EMPIRE_X_EXPOSURE which needs to be either `internal` or `internet-facing` (kept AWS terminology here).

If that environment variable is not set, we fallback to the default selection via application (where you use `domain-add`).

Fixes remind101#1076
iserko added a commit to iserko/empire that referenced this issue Jun 13, 2017
…ation

Introduces EMPIRE_X_EXPOSURE which needs to be either `internal` or `internet-facing` (kept AWS terminology here).

If that environment variable is not set, we fallback to the default selection via application (where you use `domain-add`).

Fixes remind101#1076
iserko added a commit to iserko/empire that referenced this issue Jun 13, 2017
…ation

Introduces EMPIRE_X_EXPOSURE which needs to be either `internal` or `internet-facing` (kept AWS terminology here).

If that environment variable is not set, we fallback to the default selection via application (where you use `domain-add`).

Fixes remind101#1076
iserko added a commit to iserko/empire that referenced this issue Jun 13, 2017
…ation

Introduces EMPIRE_X_EXPOSURE which needs to be either `internal` or `internet-facing` (kept AWS terminology here).

If that environment variable is not set, we fallback to the default selection via application (where you use `domain-add`).

Fixes remind101#1076
iserko added a commit to iserko/empire that referenced this issue Jun 13, 2017
…ation

Introduces EMPIRE_X_EXPOSURE which needs to be either `internal` or `internet-facing` (kept AWS terminology here).

If that environment variable is not set, we fallback to the default selection via application (where you use `domain-add`).

Fixes remind101#1076
iserko added a commit to iserko/empire that referenced this issue Aug 2, 2017
…ation

Introduces EMPIRE_X_EXPOSURE which needs to be either `internal` or `internet-facing` (kept AWS terminology here).

If that environment variable is not set, we fallback to the default selection via application (where you use `domain-add`).

Fixes remind101#1076
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

No branches or pull requests

2 participants