Right -- I'll add a note to that effect to the CHANGELOG for 1.3.0. Neil
On Fri, Jan 20, 2017 at 7:11 PM, Benjamin Mahler <[email protected]> wrote: > +1 > > Thanks for taking on the explicit rejection, much needed safety > improvement. If we state that a Mesos 1.3.0 master will not support pre-1.0 > agents, it seems to carry an implication that we supported 1.2 and 1.1 > having pre-1.0 agents. While it might work, we should clarify for users per > vinod's comment. > > On Fri, Jan 20, 2017 at 12:45 PM, Vinod Kone <[email protected]> wrote: > >> +1 >> >> Technically 0.28.0 was only supposed to be compatible with 0.27.0 and 1.0. >> >> >> On Fri, Jan 20, 2017 at 8:02 PM, Zameer Manji <[email protected]> wrote: >> >> > +1 >> > >> > >> > >> > On Fri, Jan 20, 2017 at 10:58 AM, Neil Conway <[email protected]> >> > wrote: >> > >> > > I'd like to propose that the Mesos 1.3.0 should not allow pre-1.0 >> > > Mesos agents to register. >> > > >> > > Motivation: >> > > >> > > (1) We can simplify the master code in a few places. For example, we >> > > can assume that we always have a FrameworkInfo for any task running on >> > > a registered agent. Needing to handle running tasks without a >> > > FrameworkInfo makes the code unreadable and has been a source of bugs. >> > > >> > > (2) The master only needs to report "orphan tasks" and "unregistered >> > > frameworks" if the cluster contains pre-1.0 agents. If we disallow >> > > such agents, we can remove the code for computing these fields in the >> > > HTTP endpoints and elsewhere. (We'll probably still need to keep the >> > > actual fields in the JSON/protobuf output for backward compatibility, >> > > but they will always be empty.) We can also remove "orphan tasks" from >> > > the web UI. >> > > >> > > In addition to declaring that Mesos 1.3.0 masters will not support >> > > pre-1.0 Mesos agents in the CHANGELOG, it seems safer to me to >> > > disallow such agents from registering. >> > > >> > > Comments welcome. >> > > >> > > Thanks, >> > > Neil >> > > >> > > -- >> > > Zameer Manji >> > > >> > >>
