Bug 33091 – Rewrite des setup-Skripts net/10interfaces

Bug 33091 - Rewrite des setup-Skripts net/10interfaces


Summary:	Rewrite des setup-Skripts net/10interfaces

Status:	CLOSED DUPLICATE of bug 28670

Product:	UCS
Classification:	Unclassified
Component:	System setup
Version:	UCS 3.0
Hardware:	Other Linux

Importance:	P3 enhancement (vote)
Target Milestone:	UCS 3.2
Assigned To:	UCS maintainers
QA Contact:	Stefan Gohmann

URL:
Keywords:	interim-3

Depends on:	28670 32324 32544 32565 32817 32818 32931 36561 37036 37167
Blocks:	29653 30816 30878
	Show dependency tree / graph

Reported:	2013-11-01 14:07 CET by Felix Botner
Modified:	2018-04-14 13:37 CEST (History)
CC List:	7 users (show)

See Also:
What kind of report is it?:	---
What type of bug is this?:	---
Who will be affected by this bug?:	---
How will those affected feel about the bug?:	---
User Pain:
Enterprise Customer affected?:
School Customer affected?:
ISV affected?:
Waiting Support:
Flags outvoted (downgraded) after PO Review:
Ticket number:
Bug group (optional):	Cleanup
Max CVSS v3 score:

Attachments
profile (43 bytes, text/plain) 2013-11-01 14:10 CET, Felix Botner	Details
setup.log (194.81 KB, text/x-log) 2013-11-01 14:10 CET, Felix Botner	Details
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.

Description Felix Botner

2013-11-01 14:07:51 CET

+++ This bug was initially created as a clone of Bug #28670 +++

Das Join-Skript net/10interfaces hat mittlerweile weit über 500 Zeilen Bashcode, teilweise ziemlich obskuren (mindestens mir geht es so).

Im Rahmen von Bug #26608 hätte der Code für den Fallback der IP-Adressen rausgemusst, aber es ist unklar, wieviel da dran hängt. Also bleibt es halt drin - es macht ja nichts kaputt.

Dieses Skript bedarf eines Cleanup, am besten eines Rewrite in Python. Es gibt bereits Skripte, die in Python implementiert sind (z.B. 10role), je mehr, desto besser.

Comment 1 Felix Botner

2013-11-01 14:09:51 CET

step 1:
  add ipv6 address, then apply

step 2:
  added ipv6 gateway, then apply

got:
  multiple tracebacks in setup.log
  INFO:uss.network.phase.RewriteUcr:Updating 'nameserver1'='None'
  INFO:uss.network.phase.RewriteUcr:Updating 'ucc/pxe/nameserver'='None'
  INFO:uss.network.phase.RewriteUcr:Updating 'ucc/pxe/nfsroot'='None'
  ...

Comment 2 Felix Botner

2013-11-01 14:10:12 CET

Created attachment 5550 [details]
profile

Comment 3 Felix Botner

2013-11-01 14:10:31 CET

Created attachment 5551 [details]
setup.log

Comment 4 Felix Botner

2013-11-01 14:10:44 CET

UCS 3.2 Master

Comment 5 Felix Botner

2013-11-01 14:57:28 CET

this is not a ipv6-only issue, we have to fix it for 3.2-0

*** This bug has been marked as a duplicate of bug 28670 ***

Comment 6 Philipp Hahn

2013-11-01 15:23:44 CET

(In reply to Felix Botner from comment #1)
> step 1:
>   add ipv6 address, then apply
> 
> step 2:
>   added ipv6 gateway, then apply

This is not done by 10network, but by two other scripts 12gateway and 13ipv6gateway.

> got:
>   multiple tracebacks in setup.log
>   INFO:uss.network.phase.RewriteUcr:Updating 'nameserver1'='None'
>   INFO:uss.network.phase.RewriteUcr:Updating 'ucc/pxe/nameserver'='None'
>   INFO:uss.network.phase.RewriteUcr:Updating 'ucc/pxe/nfsroot'='None'
>   ...

This seems to be a bug in 10network: The profile only contains the ipv6-gateway-setting and no interface settings at all. This situation is not detected by the script.
Easiest fix: disable any action if no interfaces/ setting is contained.

Comment 7 Stefan Gohmann

2013-11-01 20:04:03 CET

OK, duplicate

Comment 8 Stefan Gohmann

2013-11-19 06:43:18 CET

UCS 3.2 has been released:
 http://docs.univention.de/release-notes-3.2-en.html
 http://docs.univention.de/release-notes-3.2-de.html

If this error occurs again, please use "Clone This Bug".