feature flags don't propagate to windows agents

Bug #1401368 reported by Tim Penhey
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
juju-core
Fix Released
High
Bogdan Teleaga
1.24
Won't Fix
Medium
Unassigned

Bug Description

Juju feature flags are passed to the agents using an environment variable.

Getting environment variables into the service running the agents on windows does not support this currently. It needs to.

Tags: windows
Curtis Hovey (sinzui)
Changed in juju-core:
milestone: none → 1.22
Curtis Hovey (sinzui)
Changed in juju-core:
milestone: 1.22 → 1.23
Curtis Hovey (sinzui)
Changed in juju-core:
milestone: 1.23 → 1.24-alpha1
Curtis Hovey (sinzui)
Changed in juju-core:
milestone: 1.24-alpha1 → 1.24.0
Curtis Hovey (sinzui)
Changed in juju-core:
milestone: 1.24.0 → 1.25.0
Revision history for this message
Gabriel Samfira (gabriel-samfira) wrote :

Passing custom environment variables to a service is not supported on windows. This can be done either via registry keys, or using something like:

https://msdn.microsoft.com/en-us/library/windows/desktop/ms685138%28v=vs.85%29.aspx

So to achieve the goal of actually passing in feature flags, we need to abstract this into something that does the right thing for each supported platform.

Could anyone point me to where feature flags are interpreted by jujud?

Revision history for this message
Tim Penhey (thumper) wrote :

Sure,

cmd/jujud/main.go line 31

This sets the feature flags for the JUJUD executable based on the environment variable.

The env var is passed to jujud though either the upstart or systemd script which is created through the cloud init code.

What we'd need to do is have the code that generates the cloud-init config for the windows machines to set the appropriate registry keys that jujud can then access.

We could then have a common method within juju to set the feature flags, so it does the right thing based on the host OS.

Revision history for this message
Gabriel Samfira (gabriel-samfira) wrote :

SGTM. Will get on this ASAP.

Changed in juju-core:
assignee: nobody → Bogdan Teleaga (bteleaga)
Changed in juju-core:
status: Triaged → Fix Committed
Curtis Hovey (sinzui)
Changed in juju-core:
status: Fix Committed → Fix Released
Revision history for this message
Anastasia (anastasia-macmood) wrote :

targeted against old milestone

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.