← Back to team overview

sts-sponsors team mailing list archive

Thread
Date

Re: [Merge] ~ack/maas-site-manager:token-csv-export into maas-site-manager:main

To: mp+443428@xxxxxxxxxxxxxxxxxx
From: Thorsten Merten <mp+443428@xxxxxxxxxxxxxxxxxx>
Date: Tue, 23 May 2023 15:43:52 -0000
In-reply-to: <168485441117.25517.11727295289099211898.launchpad@juju-4112d9-prod-launchpad-manual-servers-36.openstack.prodstack5.lan>
Reply-to: mp+443428@xxxxxxxxxxxxxxxxxx
Sender: noreply@xxxxxxxxxxxxx

Review: Approve

lgtm!
One thing: We could also test if get_active_tokens is getting only active tokens

-- 
https://code.launchpad.net/~ack/maas-site-manager/+git/site-manager/+merge/443428
Your team MAAS Committers is subscribed to branch ~ack/maas-site-manager:token-csv-export.