Hello,

this is the method which is called (from the NIC):

      def vlanid
        # Determine a vlanid according to the following cascading rules:
        # 1. if the interface has a tag, use that as the vlanid
        # 2. if the interface has a v4 subnet with a non-blank vlanid, use that
        # 3. if the interface has a v6 subnet with a non-blank vlanid, use that
        # 4. if no reasonable vlanid was determined, then return an empty string
        #
        # In case the v4 and v6 subnet are both present, they should have the same
        # vlanid. If they have a different vlanid, this is probably an error in the
        # user's database. There is no way of determining the real vlanid, so we
        # pick the v4 one unless it turns out to be blank.

        return tag if tag.present?
        return subnet.vlanid if subnet && subnet.vlanid.present?
        return subnet6.vlanid if subnet6 && subnet6.vlanid.present?
        ''
      end

So it looks like the method does not check if the nic is virtual or not, I agree with your assesment. Feel free to contribute a change via github! Cheers.