Bug 791044

Summary:	deleting an entry from libra-proxy deletes all entries after it
Product:	OKD	Reporter:	Rob Millner <rmillner>
Component:	Containers	Assignee:	Rob Millner <rmillner>
Status:	CLOSED CURRENTRELEASE	QA Contact:	libra bugs <libra-bugs>
Severity:	high	Docs Contact:
Priority:	high
Version:	2.x	CC:	lilu, mfisher, xtian
Target Milestone:	---	Keywords:	Triaged
Target Release:	---
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:	rhc-node-0.86.6-1+	Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2012-02-17 15:50:32 UTC	Type:	---
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:

Description Rob Millner 2012-02-16 01:06:05 UTC

Description of problem:

When removing a proxy from libra-proxy in the middle of the configuration, all proxies after it in the file are removed as well.

Version-Release number of selected component (if applicable):


How reproducible:
Always

Steps to Reproduce:
1. Create 5 apps and do an expose-port on all of them
2. Test proxies to ensure that they are working
3. Pick the first app and do a conceal-port on it
4. Test the remaining 4 proxies
  
Actual results:

All remaining proxies fail

Expected results:

All remaining proxies still work

Additional info:

The bug lies in the libra-proxy init script, setproxy function.

Comment 1 Rob Millner 2012-02-16 01:18:27 UTC

Missing space caused sed expression to match till the end of the file.

Commit: 8f8b788

Comment 2 Xiaoli Tian 2012-02-16 08:12:20 UTC

(In reply to comment #1)
> Missing space caused sed expression to match till the end of the file.
> 
> Commit: 8f8b788

Have checked the latest instances like devenv-stage_132 and devenv_1613,your fix "Commit: 8f8b788 " for this bug has not been built in .

But after changing the code in the sever directly according to your fix, it works,
We'll wait for next build come to verify it again.

Comment 3 Linqing Lu 2012-02-17 07:59:28 UTC

verified it's working on devenv_1616