Bug 676864

Summary:	pulp-client consumer doesn't check for pre-existing consumer
Product:	[Retired] Pulp	Reporter:	dgao
Component:	user-experience	Assignee:	Pradeep Kilambi <pkilambi>
Status:	CLOSED DUPLICATE	QA Contact:	Preethi Thomas <pthomas>
Severity:	unspecified	Docs Contact:
Priority:	unspecified
Version:	unspecified	CC:	skarmark
Target Milestone:	---	Keywords:	Triaged
Target Release:	---
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:		Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2011-10-10 18:23:01 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 dgao 2011-02-11 15:56:00 UTC

[root@pulp-qe ~]# pulp-admin -u admin -p admin auth login
User credentials successfully stored at [/root/.pulp]

[root@pulp-qe ~]# pulp-admin repo create --id=foo --feed=yum:http://10.16.76.78/pub/updates
Successfully created repository [ foo ]

[root@pulp-qe ~]# pulp-admin repo sync --id=foo -F
Sync for repository foo started
Sync: Finished
0/6 new items downloaded
6/6 existing items processed

Item Details: 
Rpms: 6/6

[root@pulp-qe ~]# pulp-client consumer create --id fred
warning: this client is not registered; please register to continue
Successfully created consumer [ fred ]

[root@pulp-qe ~]# pulp-client  consumer bind --repoid foo
Successfully subscribed consumer [fred] to repo [foo]

[root@pulp-qe ~]# pulp-client consumer create --id ethel
Successfully created consumer [ ethel ]

[root@pulp-qe ~]# pulp-client consumer delete --id ethel
Successfully deleted consumer [ethel]

[root@pulp-qe ~]# pulp-client  consumer unbind --repoid foo
warning: this client is not registered; please register to continue
Usage: pulp-client <options> consumer unbind <options>

pulp-client: error: Option --id is required; please see --help
[root@pulp-qe ~]# pulp-client  consumer delete
warning: this client is not registered; please register to continue
Usage: pulp-client <options> consumer delete <options>

pulp-client: error: Option --id is required; please see --help
[root@pulp-qe ~]# ls /etc/pki/consumer/
[root@pulp-qe ~]#

Because the consumer cert is gone, pulp-client doesn't remember the previous client and the repo it bound to. As a result, the only way to clean up this mess is to use the pulp-admin CLI to delete the consumer. 

There should be a check during pulp-client consumer create for pre-existing consumer.

Comment 1 Sayli Karmarkar 2011-07-15 06:26:48 UTC

This should be fixed.

Comment 2 dgao 2011-07-15 15:22:52 UTC

[root@pulp-qe ~]# pulp-admin repo create --id=foo --feed=http://10.16.76.78/pub/updates
Successfully created repository [ foo ]

[root@pulp-qe ~]# pulp-admin repo sync --id=foo -F
Sync for repository foo started
Sync: Finished
0/7 new items downloaded
7/7 existing items processed

Item Details: 
Rpms: 7/7

[root@pulp-qe ~]# pulp-client -u admin -p admin consumer create --id fred
Successfully created consumer [ fred ]

[root@pulp-qe ~]# pulp-client  consumer bind --repoid foo
Successfully subscribed consumer [fred] to repo [foo]

[root@pulp-qe ~]# pulp-client consumer create --id ethel
Successfully created consumer [ ethel ]

[root@pulp-qe ~]# pulp-admin consumer delete --id ethel
Successfully deleted consumer [ethel]

[root@pulp-qe ~]# pulp-client  consumer unbind --repoid foo
error: operation failed: Invalid SSL Certificate
[root@pulp-qe ~]# ls /etc/pki/consumer/
cert.pem

Comment 3 dgao 2011-07-15 15:24:06 UTC

[root@pulp-qe ~]# openssl x509 -in /etc/pki/consumer/cert.pem -text
Certificate:
    Data:
        Version: 1 (0x0)
        Serial Number: 16 (0x10)
        Signature Algorithm: sha1WithRSAEncryption
        Issuer: CN=localhost
        Validity
            Not Before: Jul 15 15:13:48 2011 GMT
            Not After : Jul 12 15:13:48 2021 GMT
        Subject: CN=ethel
        Subject Public Key Info:
            Public Key Algorithm: rsaEncryption
                Public-Key: (1024 bit)
                Modulus:
                    00:d9:47:8f:e0:0f:8a:e9:e3:bc:90:0d:b4:fb:d7:
                    07:3f:7e:ca:be:3b:30:d5:3e:a3:c0:6d:b4:0d:2d:
                    cd:42:15:bf:8a:0b:23:e4:d9:54:02:22:4e:a1:aa:
                    27:18:5f:dd:ba:a0:94:50:81:d8:96:49:0a:07:2a:
                    f6:e4:d2:83:9c:ca:8c:5c:5f:04:d8:92:a2:96:91:
                    9a:c7:32:d3:5c:6f:c3:7d:f6:4c:bc:3b:b3:a2:27:
                    6f:69:29:9a:d1:a7:a0:85:88:0d:af:f4:11:c3:20:
                    b4:d4:f4:65:f9:5c:8d:82:83:ba:78:4c:51:e7:89:
                    21:07:ea:d2:df:f5:c0:69:6d
                Exponent: 65537 (0x10001)
    Signature Algorithm: sha1WithRSAEncryption
        12:80:55:af:12:9c:04:ec:89:1d:80:11:c1:ee:56:8e:cb:47:
        c1:7e:e7:9f:0b:6c:36:ca:1c:f2:82:65:b2:2e:d8:37:52:56:
        61:bd:63:b8:da:4d:9d:fd:d0:71:0a:2a:a2:1f:5c:86:12:c4:
        c6:23:ed:ac:70:aa:41:f8:f2:d9:48:70:e2:6f:18:c5:d4:6f:
        4a:86:11:70:f5:b0:6e:63:50:b1:5a:7c:ef:13:38:7b:59:24:
        d8:d8:ad:4b:d2:86:25:10:92:f9:71:58:bd:29:6e:a2:85:75:
        91:34:3e:cc:fa:4e:7d:9c:4e:cb:13:1f:02:8d:33:b3:dc:5d:
        17:9d:c8:46:d9:30:af:0d:78:b0:3c:b1:a1:da:7d:72:88:28:
        45:bb:a7:31:45:fc:a5:1f:74:64:b6:a0:a1:5e:db:95:c6:ae:
        25:73:1c:72:f5:59:d9:12:b0:e8:2f:00:76:4a:1e:69:be:54:
        44:0b:56:fd:af:1f:0c:5c:68:44:a9:d2:57:66:2b:36:53:dd:
        7d:62:c5:23:fa:ab:71:ea:c9:1d:ff:8a:6f:24:d3:a5:c7:f4:
        bd:9c:00:51:85:a2:54:e0:ea:78:09:e6:ad:0f:05:3b:9f:cd:
        9b:ef:0c:1a:17:77:a4:fe:44:f2:59:c0:53:a7:7c:62:00:d6:
        df:a4:7d:59
-----BEGIN CERTIFICATE-----
MIICEzCB/AIBEDANBgkqhkiG9w0BAQUFADAUMRIwEAYDVQQDEwlsb2NhbGhvc3Qw
HhcNMTEwNzE1MTUxMzQ4WhcNMjEwNzEyMTUxMzQ4WjAQMQ4wDAYDVQQDEwVldGhl
bDCBnzANBgkqhkiG9w0BAQEFAAOBjQAwgYkCgYEA2UeP4A+K6eO8kA20+9cHP37K
vjsw1T6jwG20DS3NQhW/igsj5NlUAiJOoaonGF/duqCUUIHYlkkKByr25NKDnMqM
XF8E2JKilpGaxzLTXG/DffZMvDuzoidvaSma0aeghYgNr/QRwyC01PRl+VyNgoO6
eExR54khB+rS3/XAaW0CAwEAATANBgkqhkiG9w0BAQUFAAOCAQEAEoBVrxKcBOyJ
HYARwe5WjstHwX7nnwtsNsoc8oJlsi7YN1JWYb1juNpNnf3QcQoqoh9chhLExiPt
rHCqQfjy2Uhw4m8YxdRvSoYRcPWwbmNQsVp87xM4e1kk2NitS9KGJRCS+XFYvSlu
ooV1kTQ+zPpOfZxOyxMfAo0zs9xdF53IRtkwrw14sDyxodp9cogoRbunMUX8pR90
ZLagoV7blcauJXMccvVZ2RKw6C8Adkoeab5URAtW/a8fDFxoRKnSV2YrNlPdfWLF
I/qrcerJHf+KbyTTpcf0vZwAUYWiVODqeAnmrQ8FO5/Nm+8MGhd3pP5E8lnAU6d8
YgDW36R9WQ==
-----END CERTIFICATE-----


Looks like pulp removed the wrong cert when I deleted ethel.

Comment 4 Pradeep Kilambi 2011-10-10 18:23:01 UTC


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