You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
When this step was added to blueprint execution, it was required so sled-agent could get firewall rules for newly-deployed Nexus instances. Since then, the ServiceRulePropagator RPW has been added, which plumbs firewall rules periodically.
I think it would be fine to just delete the blueprint execution step entirely, but needs at least some basic testing to ensure that's right. The activation period for ServiceRulePropagator is currently pretty long (5 minutes), so there might be cases for:
shorten the ServiceRulePropagator period
have blueprint execution activate ServiceRulePropagator explicitly after deploying zones (I think this would be equivalent to the previous bullet? since we'd now be activating ServiceRulePropagator on the blueprint-execution period)
have blueprint execution activate ServiceRulePropagator explicitly after deploying zones if any new zones were added (I think this would require more information from sled-agent about what it did to handle a PUT /omicron-zones request)
The text was updated successfully, but these errors were encountered:
When this step was added to blueprint execution, it was required so sled-agent could get firewall rules for newly-deployed Nexus instances. Since then, the
ServiceRulePropagator
RPW has been added, which plumbs firewall rules periodically.I think it would be fine to just delete the blueprint execution step entirely, but needs at least some basic testing to ensure that's right. The activation period for
ServiceRulePropagator
is currently pretty long (5 minutes), so there might be cases for:ServiceRulePropagator
periodServiceRulePropagator
explicitly after deploying zones (I think this would be equivalent to the previous bullet? since we'd now be activatingServiceRulePropagator
on the blueprint-execution period)ServiceRulePropagator
explicitly after deploying zones if any new zones were added (I think this would require more information from sled-agent about what it did to handle aPUT /omicron-zones
request)The text was updated successfully, but these errors were encountered: