fixes #330 #351

Closed
wants to merge 4 commits into
from

Projects

None yet

4 participants

@borgified
Contributor

im guessing this is what you mean by adding the security group id

@treeder
Member
treeder commented Sep 26, 2013

The security group id is simple_worker_sg (i believe).

@featalion
Contributor

A lot of people asked about security group string already defined at DevCenter

@borgified
Contributor

Ah... didn't realize similar information existed in two places:

http://dev.iron.io/worker/reference/security/#security_groups_and_ip_ranges
http://dev.iron.io/worker/reference/environment/#security_groups_and_ip_ranges

The 1st link mentions: "Please note that this security group only works in the US regions."

Should the information about security_groups_and_ip_ranges in 2nd link be copied over to the first link?
Should the note about only working in the US regions be included in the 2nd link?

@featalion
Contributor

I assume we can remove section at second link, but definitely need link to security page on navigation menu in section Reference.

@borgified
Contributor

awaiting confirmation of removing security_groups_and_ip_ranges section at second link.
correct to assume 2nd link has the "better/more current" information?

@carimura
Member

cc @featalion

hey @borgified send an address to chad[at]iron.io and we'll get you off a t-shirt!

@featalion
Contributor

Maybe good to keep it in both sections, but add the link to security section at environment section.
I will do.

@featalion featalion referenced this pull request Sep 27, 2013
Closed

Improvements #355

@featalion
Contributor

merged and improved, #355

@featalion featalion closed this Sep 27, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment