[node-devel] New networking setup stuff

[Fabian Deutsch]

Am Montag, den 17.06.2013, 15:42 -0400 schrieb Mike Burns:
> On 06/17/2013 03:34 PM, Fabian Deutsch wrote:
> > Am Montag, den 17.06.2013, 13:06 -0400 schrieb Mike Burns:
> >> Things look good so far, but some questions/issues that I found:
> >>
> >> Questions:
> >> Is there a way to toggle bridged/direct in the TUI?
> >
> > Yep, I actually don't know the patch - maybe I need to reorganize it -
> > but it's set in the NIC Details configuration, "Use bridge?   [ ]"
> 
> Unless i missed the patch, I'm not seeing this.  I did checkout of this 
> patch, rebased on top of latest git, and built:
> 
> http://gerrit.ovirt.org/#/c/15700/

I fixed it up in one of the bond related patches.

> 
> >
> >>
> >> Issues:
> >>
> >> Error when de-activating nic with no bridge:
> >>     3/5 Write new configuration
> >> An error occurred while applying the changes: udev has no infos for None
> >
> > Yep, also ran into that one.
> >
> >> Nic Config transaction says "DNS and NTP configuration"  this might be
> >> old/existing functionality
> >
> > Yep, renamed it to "Basic Network Configuration", or maybe some other
> > wording?
> 
> NIC Configuration or Network Interface Configuration

ack

> >
> >> Status screen shows "Bridge: eth0"
> >
> > Yep, I removed the hard coded "Bridge", now geneirc network informations
> > are shown.
> >
> 
> ACK
> 
> >> not sure when this showed up so might be unrelated -- ovirt-node.log in
> >> /var/log
> >
> > Yes, it should be there.
> 
> Can you tell me what is in which log?
> 
> ovirt.log
> ovirt-debug.log
> ovirt-node.log
> 
> And can/should we consolidate?

Ah, now I see your point.
ovirt.log - our legacy logfile, it will stay around until we migrated 
            all legacy code.

ovirt-debug.log - debug level logger
ovirt-node.log - info level logger (much much less informations)

On the long run ovirt.log and ovirt-node.log will be "merged",
ovirt-node.debug.log can be disabled, maybe we can add a fag to enable
it ..

Fabian

> 
> Mike
> >
> > Thanks for looking at it and your comments! I'll fix those issue in
> > upcoming commits.
> >
> > - fabian
> >
>