package-import uses james_w credentials

Bug #524173 reported by James Westby
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Ubuntu Distributed Development
Fix Released
High
Martin Pool

Bug Description

package-import currently poses as me so that it can get the privileges I have, which
was ok when I was the only one with access, but we should make it such that it runs
as a new identity.

We need to:

  * create a new LP user.
  * make that user part of ~ubuntu-branches so that it can set official branches.
  * allow that user to write to any official branch.

The last part isn't pretty, but I don't see a way around it given that not all branches
are going to be owned by ~ubuntu-branches.

Thanks,

James

Revision history for this message
Robert Collins (lifeless) wrote :

We should also get the sysadmins (e.g. losas) access to that account - ideally they will set it up like they did bzr-pqm etc.

summary: - package-import uses my credentials
+ package-import uses james_w credentials
Vincent Ladeuil (vila)
Changed in udd:
assignee: nobody → canonical-bazaar (canonical-bazaar)
Revision history for this message
Martin Pool (mbp) wrote :

> * create a new LP user.
> * make that user part of ~ubuntu-branches so that it can set official branches.
> * allow that user to write to any official branch.

Is the third of these a consequence of the second, or does something else need to be done?

Revision history for this message
James Westby (james-w) wrote : Re: [Bug 524173] Re: package-import uses james_w credentials

On Wed, 02 Feb 2011 02:26:37 -0000, Martin Pool <email address hidden> wrote:
> > * create a new LP user.
> > * make that user part of ~ubuntu-branches so that it can set official branches.
> > * allow that user to write to any official branch.
>
> Is the third of these a consequence of the second, or does something
> else need to be done?

It's almost a consequence given that the majority of official branches
are owned by that team.

However, we have been relying on the fact that I am a core-dev to allow
us to push to all branches.

Thanks,

James

Revision history for this message
Martin Pool (mbp) wrote :

On 3 February 2011 01:39, James Westby <email address hidden> wrote:
> On Wed, 02 Feb 2011 02:26:37 -0000, Martin Pool <email address hidden> wrote:
>> >  * create a new LP user.
>> >  * make that user part of ~ubuntu-branches so that it can set official branches.
>> >  * allow that user to write to any official branch.
>>
>> Is the third of these a consequence of the second, or does something
>> else need to be done?
>
> It's almost a consequence given that the majority of official branches
> are owned by that team.
>
> However, we have been relying on the fact that I am a core-dev to allow
> us to push to all branches.

So would making the bot account be part of ~core-dev be a good idea
too, and would that be sufficient?

Revision history for this message
Colin Watson (cjwatson) wrote :

I'd really prefer not. The bot should only be able to modify branches,
not trigger uploads to the archive. I thought we made ~ubuntu-branches
a celebrity so that it could do this kind of thing?

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

On Wed, 02 Feb 2011 23:37:15 -0000, Colin Watson <email address hidden> wrote:
> I'd really prefer not. The bot should only be able to modify branches,
> not trigger uploads to the archive. I thought we made ~ubuntu-branches
> a celebrity so that it could do this kind of thing?

I think that is what we should do as well.

I think the current state is that it just has special permission for
setting official branches, and not for pushing to them.

I guess it would be easy to extend it though.

Thanks,

James

Revision history for this message
Martin Pool (mbp) wrote :

Francis is now in the process of removing ~ubuntu-branches, so the options are now
 * add to ~core-dev which is said above to be undesirable
 * or, add it to the "uploaders" in <https://launchpad.net/ubuntu/> which is a member of ~techboard can do by running an api script (I don't know the name)

Revision history for this message
Francis J. Lacoste (flacoste) wrote :

We are removing the Ubuntu branches celebrity. Setting the official package branch will be controlled through the normal upload permission.

So I think this argue either for adding a robot to Ubuntu Core Devs, or granting archive upload permission through the robot directly (through the archive permissions API).

Also, once build-from-branch-into-main is complete, there won't be any real differences between uploading to a branch and to the archive.

Martin Pool (mbp)
Changed in udd:
assignee: canonical-bazaar (canonical-bazaar) → Martin Pool (mbp)
Revision history for this message
Martin Pool (mbp) wrote :
Revision history for this message
Martin Pool (mbp) wrote :

rt 46239 for creating a mail alias.

Martin Pool (mbp)
Changed in udd:
status: Triaged → In Progress
Revision history for this message
Martin Pool (mbp) wrote :

https://launchpad.net/~package-import now exists and has an ssh key. Next, according to the prior discussion, we need to add it to ~core-dev.

Changed in udd:
importance: Critical → High
Revision history for this message
Martin Pool (mbp) wrote :

That robot account is now also in launchpad.net/~ubuntu-core-dev so now we just need to switch the actual machine.

I'm going to leave that for tomorrow when I have more uninterrupted time to deal with any possible fallout.

Revision history for this message
Martin Pool (mbp) wrote :

This is switched over, and I think it's all working. We need to check a bit for fallout in the next day or so.

Changed in udd:
status: In Progress → Fix Released
Revision history for this message
Martin Pool (mbp) wrote :

I omitted to change the lp api token; will do that now.

Changed in udd:
status: Fix Released → In Progress
Revision history for this message
Ubuntu Package Importer (package-import) wrote :

The lp token is now switched. James, if you want (and if you can identify it) you can disable the old token from your account.

for the record, I switched it by moving away the old credential file, and running

udd.lpapi.get_lp()

Changed in udd:
status: In Progress → Fix Released
Revision history for this message
Max Bowsher (maxb) wrote :

So you didn't actually need to use create_creds.py in this case, then? In that case, let's delete that script.

Also, there are still various references to James in the settings of BZR_EMAIL in various udd scripts, so reopening bug.

Changed in udd:
status: Fix Released → Confirmed
Revision history for this message
Martin Pool (mbp) wrote :

That's addressed in lp:~/udd/524173-script which is currently present but uncommitted on Jubany.

Revision history for this message
Martin Pool (mbp) wrote :
Revision history for this message
Martin Pool (mbp) wrote :

I _think_ this is now done; if not please reopen again.

Changed in udd:
status: Confirmed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.