Canonical SSO provider

Merge lp://qastaging/~jml/canonical-identity-provider/clean-up-temp into lp://qastaging/canonical-identity-provider/release

  1. clean-up-temp
  2. Merge into trunk
Proposed by Jonathan Lange
Status: Merged
Approved by: Jonathan Lange
Approved revision: no longer in the source branch.
Merged at revision: 616
Proposed branch: lp://qastaging/~jml/canonical-identity-provider/clean-up-temp
Merge into: lp://qastaging/canonical-identity-provider/release
Diff against target: 70 lines (+41/-0)
2 files modified
scripts/run-acceptance-tests (+20/-0)
tarmac_tests.sh (+21/-0)
To merge this branch: bzr merge lp://qastaging/~jml/canonical-identity-provider/clean-up-temp
Reviewer Review Type Date Requested Status
Natalia Bidart (community) Approve
James Westby (community) Approve
Review via email: mp+143537@code.qastaging.launchpad.net

Commit message

Control the creation and deletion of /tmp/djopenid_consumer_store

Description of the change

Changes the tarmac_tests.sh and run-acceptance-tests scripts to control
the creation and deletion of /tmp/djopenid_consumer_store, which is used
by webui for storing openid consumer information.

The problem is that it gets created implicitly when webui gets a startOpenID
or finishOpenID request, and when it does, it is owned by whichever user
does this.

Thus, this changes the test scripts at least to make sure it's owned by the
'tarmac' group and that anyone in the group can at least clear out the
contents.

To post a comment you must log in.
Revision history for this message
Natalia Bidart (nataliabidart) wrote :

14:31 < nessita> jml: I'm looking at https://code.launchpad.net/~jml/canonical-identity-provider/clean-up-temp/+merge/143537, I was wondering if we can somehow avoid the duplication
                 of the code

review: Needs Information
Revision history for this message
James Westby (james-w) wrote :

Hi,

This looks ok to me, though I'm a little confused about why it's ok to remove the
dir at the end if it isn't at the beginning.

Natalia had a comment on IRC:

<nessita> jml: I'm looking at https://code.launchpad.net/~jml/canonical-identity-provider/clean-up-temp/+merge/143537, I was wondering if we can somehow avoid the duplication of the code

Thanks,

James

review: Approve
Revision history for this message
Jonathan Lange (jml) wrote :

On Wed, Jan 16, 2013 at 11:28 PM, James Westby
<email address hidden> wrote:
> Review: Approve
>
> Hi,
>
> This looks ok to me, though I'm a little confused about why it's ok to remove the
> dir at the end if it isn't at the beginning.
>

That was me being confused. Fixed.

> Natalia had a comment on IRC:
>
> <nessita> jml: I'm looking at https://code.launchpad.net/~jml/canonical-identity-provider/clean-up-temp/+merge/143537, I was wondering if we can somehow avoid the duplication of the code
>

* We could write a couple of shell functions and source the file.
That would still require us to duplicate the structure, but would mean
slightly less duplicated code

* We could come up with something fundamentally different.

I'm really disinclined to do the first.

jml

Revision history for this message
Ubuntu One Server Tarmac Bot (ubuntuone-server-tarmac) wrote :

Voting does not meet specified criteria. Required: Approve >= 1, Disapprove == 0, Needs Fixing == 0, Needs Information == 0, Resubmit == 0, Pending == 0. Got: 1 Approve, 1 Needs Information.

Revision history for this message
Natalia Bidart (nataliabidart) wrote :

Fair enough, approving.

review: Approve
Revision history for this message
Ubuntu One Server Tarmac Bot (ubuntuone-server-tarmac) wrote :

The attempt to merge lp:~jml/canonical-identity-provider/clean-up-temp into lp:canonical-identity-provider failed. Below is the output from the failed tests.

./tarmac_tests.sh: 10: ./tarmac_tests.sh: [-d: not found
chgrp: changing group of `/tmp/djopenid_consumer_store': Operation not permitted

Revision history for this message
Ubuntu One Server Tarmac Bot (ubuntuone-server-tarmac) wrote :

The attempt to merge lp:~jml/canonical-identity-provider/clean-up-temp into lp:canonical-identity-provider failed. Below is the output from the failed tests.

rm: cannot remove `/tmp/djopenid_consumer_store/associations/http-localhost_3A16418-Vh0ConNmIqGOigGmK.Mc00BOquk-_dwT2ZqvZFHEYRQyRgJe733FXkc': Permission denied
rm: cannot remove `/tmp/djopenid_consumer_store/associations/http-localhost_3A16544-hZZgJmFgbPHilxfRyYrgzy4G.sA-vnuzwQDpYrcMNDU4mnTFdSrEFBQ': Permission denied
rm: cannot remove `/tmp/djopenid_consumer_store/nonces/50f7461d-http-localhost_3A16418-Vh0ConNmIqGOigGmK.Mc00BOquk-4RLxV0IAVc27Z4NI.abxwzU7eH8': Permission denied
rm: cannot remove `/tmp/djopenid_consumer_store/nonces/50f741d8-http-localhost_3A16418-Vh0ConNmIqGOigGmK.Mc00BOquk-62XAfFo4G1fJhy4XhyGfpPNI_Aw': Permission denied
rm: cannot remove `/tmp/djopenid_consumer_store/nonces/50f6b1bd-http-localhost_3A16544-hZZgJmFgbPHilxfRyYrgzy4G.sA-XWcEKQ3ErjqQ3OAGU.GcuHLSnO4': Permission denied
rm: cannot remove `/tmp/djopenid_consumer_store/nonces/50f6b1c5-http-localhost_3A16544-hZZgJmFgbPHilxfRyYrgzy4G.sA-qil8w_hsfI3Lz6XX1kfP5Zp.qPU': Permission denied
rm: cannot remove `/tmp/djopenid_consumer_store/nonces/50f741d0-http-localhost_3A16418-Vh0ConNmIqGOigGmK.Mc00BOquk-OqZkNnb8g4wU2VzJHGWpu1DpaVk': Permission denied
rm: cannot remove `/tmp/djopenid_consumer_store/nonces/50f6b1b5-http-localhost_3A16544-hZZgJmFgbPHilxfRyYrgzy4G.sA-M05K1MSX6s6ffkV5lwgjuis.lOA': Permission denied
rm: cannot remove `/tmp/djopenid_consumer_store/nonces/50f741e7-http-localhost_3A16418-Vh0ConNmIqGOigGmK.Mc00BOquk-aVEjEygZlMlHo1y_HxIOrvgHWxs': Permission denied
rm: cannot remove `/tmp/djopenid_consumer_store/nonces/50f6b1ac-http-localhost_3A16544-hZZgJmFgbPHilxfRyYrgzy4G.sA-mkn2y6v98L9EXqClqk4hjHR4aZI': Permission denied
rm: cannot remove `/tmp/djopenid_consumer_store/nonces/50f741be-http-localhost_3A16418-Vh0ConNmIqGOigGmK.Mc00BOquk-Xp1OmfbzFAVsGCt4.ricO9BQVhU': Permission denied
rm: cannot remove `/tmp/djopenid_consumer_store/nonces/50f741e0-http-localhost_3A16418-Vh0ConNmIqGOigGmK.Mc00BOquk-QzMSsEI32C65Afj7si2dKoRqauE': Permission denied
rm: cannot remove `/tmp/djopenid_consumer_store/nonces/50f6b1d4-http-localhost_3A16544-hZZgJmFgbPHilxfRyYrgzy4G.sA-QFyeJTs8nh3KaW189ixTe_ykbaE': Permission denied
rm: cannot remove `/tmp/djopenid_consumer_store/nonces/50f6b1dc-http-localhost_3A16544-hZZgJmFgbPHilxfRyYrgzy4G.sA-j_clEpqQhwLqSyccghkM6zQDiRQ': Permission denied
rm: cannot remove `/tmp/djopenid_consumer_store/nonces/50f6b601-http-localhost_3A16544-hZZgJmFgbPHilxfRyYrgzy4G.sA-rDnGe.tWHgAbtthKDaLD0B7NJbA': Permission denied
rm: cannot remove `/tmp/djopenid_consumer_store/nonces/50f6b1cd-http-localhost_3A16544-hZZgJmFgbPHilxfRyYrgzy4G.sA-fHWcxwVs7yd26sZNPIpzXHreJC8': Permission denied
rm: cannot remove `/tmp/djopenid_consumer_store/nonces/50f741f0-http-localhost_3A16418-Vh0ConNmIqGOigGmK.Mc00BOquk-GKdaLqUvThenfw51ylTq2WDEBcY': Permission denied
rm: cannot remove `/tmp/djopenid_consumer_store/nonces/50f741c7-http-localhost_3A16418-Vh0ConNmIqGOigGmK.Mc00BOquk-ly0OZnWFRzxd0f.hAK6AyHgts9w': Permission denied
rm: cannot remove `/tmp/djopenid_consumer_store/temp': Permission denied

Revision history for this message
Ubuntu One Server Tarmac Bot (ubuntuone-server-tarmac) wrote :

The attempt to merge lp:~jml/canonical-identity-provider/clean-up-temp into lp:canonical-identity-provider failed. Below is the output from the failed tests.

rm: cannot remove `/tmp/djopenid_consumer_store/associations/http-localhost_3A16418-Vh0ConNmIqGOigGmK.Mc00BOquk-_dwT2ZqvZFHEYRQyRgJe733FXkc': Permission denied
rm: cannot remove `/tmp/djopenid_consumer_store/associations/http-localhost_3A16544-hZZgJmFgbPHilxfRyYrgzy4G.sA-vnuzwQDpYrcMNDU4mnTFdSrEFBQ': Permission denied
rm: cannot remove `/tmp/djopenid_consumer_store/nonces/50f7461d-http-localhost_3A16418-Vh0ConNmIqGOigGmK.Mc00BOquk-4RLxV0IAVc27Z4NI.abxwzU7eH8': Permission denied
rm: cannot remove `/tmp/djopenid_consumer_store/nonces/50f741d8-http-localhost_3A16418-Vh0ConNmIqGOigGmK.Mc00BOquk-62XAfFo4G1fJhy4XhyGfpPNI_Aw': Permission denied
rm: cannot remove `/tmp/djopenid_consumer_store/nonces/50f6b1bd-http-localhost_3A16544-hZZgJmFgbPHilxfRyYrgzy4G.sA-XWcEKQ3ErjqQ3OAGU.GcuHLSnO4': Permission denied
rm: cannot remove `/tmp/djopenid_consumer_store/nonces/50f6b1c5-http-localhost_3A16544-hZZgJmFgbPHilxfRyYrgzy4G.sA-qil8w_hsfI3Lz6XX1kfP5Zp.qPU': Permission denied
rm: cannot remove `/tmp/djopenid_consumer_store/nonces/50f741d0-http-localhost_3A16418-Vh0ConNmIqGOigGmK.Mc00BOquk-OqZkNnb8g4wU2VzJHGWpu1DpaVk': Permission denied
rm: cannot remove `/tmp/djopenid_consumer_store/nonces/50f6b1b5-http-localhost_3A16544-hZZgJmFgbPHilxfRyYrgzy4G.sA-M05K1MSX6s6ffkV5lwgjuis.lOA': Permission denied
rm: cannot remove `/tmp/djopenid_consumer_store/nonces/50f741e7-http-localhost_3A16418-Vh0ConNmIqGOigGmK.Mc00BOquk-aVEjEygZlMlHo1y_HxIOrvgHWxs': Permission denied
rm: cannot remove `/tmp/djopenid_consumer_store/nonces/50f6b1ac-http-localhost_3A16544-hZZgJmFgbPHilxfRyYrgzy4G.sA-mkn2y6v98L9EXqClqk4hjHR4aZI': Permission denied
rm: cannot remove `/tmp/djopenid_consumer_store/nonces/50f741be-http-localhost_3A16418-Vh0ConNmIqGOigGmK.Mc00BOquk-Xp1OmfbzFAVsGCt4.ricO9BQVhU': Permission denied
rm: cannot remove `/tmp/djopenid_consumer_store/nonces/50f741e0-http-localhost_3A16418-Vh0ConNmIqGOigGmK.Mc00BOquk-QzMSsEI32C65Afj7si2dKoRqauE': Permission denied
rm: cannot remove `/tmp/djopenid_consumer_store/nonces/50f6b1d4-http-localhost_3A16544-hZZgJmFgbPHilxfRyYrgzy4G.sA-QFyeJTs8nh3KaW189ixTe_ykbaE': Permission denied
rm: cannot remove `/tmp/djopenid_consumer_store/nonces/50f6b1dc-http-localhost_3A16544-hZZgJmFgbPHilxfRyYrgzy4G.sA-j_clEpqQhwLqSyccghkM6zQDiRQ': Permission denied
rm: cannot remove `/tmp/djopenid_consumer_store/nonces/50f6b601-http-localhost_3A16544-hZZgJmFgbPHilxfRyYrgzy4G.sA-rDnGe.tWHgAbtthKDaLD0B7NJbA': Permission denied
rm: cannot remove `/tmp/djopenid_consumer_store/nonces/50f6b1cd-http-localhost_3A16544-hZZgJmFgbPHilxfRyYrgzy4G.sA-fHWcxwVs7yd26sZNPIpzXHreJC8': Permission denied
rm: cannot remove `/tmp/djopenid_consumer_store/nonces/50f741f0-http-localhost_3A16418-Vh0ConNmIqGOigGmK.Mc00BOquk-GKdaLqUvThenfw51ylTq2WDEBcY': Permission denied
rm: cannot remove `/tmp/djopenid_consumer_store/nonces/50f741c7-http-localhost_3A16418-Vh0ConNmIqGOigGmK.Mc00BOquk-ly0OZnWFRzxd0f.hAK6AyHgts9w': Permission denied
rm: cannot remove `/tmp/djopenid_consumer_store/temp': Permission denied

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.