| * tomwardill pokes buildbot a bit more | 07:52 | |
| * ilasc is writing a book for the Support Wiki with answers she's getting from wgrant this week :) | 07:57 | |
| tomwardill | excellent :) | 07:59 |
|---|---|---|
| ilasc | tomwardill: just checking I understood correctly from the buildbot emails I saw this morning, your Store Username merged & built successfully, then API for OCI Registry Credentials failed with the usual and you triggered a forced build which again failed with the usual ? | 08:07 |
| tomwardill | yep | 08:07 |
| tomwardill | I've just triggered another one | 08:07 |
| tomwardill | they were slightly different usuals each time, but in that group, yeah | 08:08 |
| ilasc | ok cool, thanks! :) | 08:08 |
| ilasc | tomwardill: code-wise this means I can now rebase / merge my UI branch to take in the Username changes from master | 08:11 |
| tomwardill | yep, it should be merged | 08:11 |
| ilasc | ok, great | 08:13 |
| cjwatson | tomwardill: So if I upgrade dogfood, it should be possible to run an end-to-end test? | 08:32 |
| tomwardill | errr, I think so? :) | 08:33 |
| tomwardill | does dogfood run upload jobs automatically? | 08:33 |
| cjwatson | We'd need to run that manually | 08:33 |
| tomwardill | right | 08:33 |
| tomwardill | yeah, I think we should be good to try it | 08:34 |
| wgrant | ilasc: Thanks for continuing to write the manual that we should have written years ago :) | 08:38 |
| cjwatson | tomwardill: dogfood is current now | 08:46 |
| tomwardill | well | 08:46 |
| tomwardill | better try a thing then :) | 08:46 |
| tomwardill | huh, getOCIProject isn't exported on distribution | 08:49 |
| * tomwardill takes notes | 08:52 | |
| tomwardill | https://dogfood.paddev.net/~twom/ubuntu/+oci/test-twom-2020-04-22-2/+recipe/test-twom-2020-04-22-2-recipe/+build/7 | 08:53 |
| * tomwardill forgot to add any credentials to the recipe | 08:56 | |
| * tomwardill does that | 08:56 | |
| tomwardill | in lp-shell, is there a way to load an object from an arbitary API call? | 08:59 |
| cjwatson | Like lp.load you mean? | 08:59 |
| tomwardill | oh, maybe that :) | 08:59 |
| cjwatson | Hm, no OCIRecipeBuildJob created by your build. | 09:00 |
| cjwatson | Ah, maybe you haven't actually created the push rule yet? | 09:01 |
| tomwardill | no, I failed | 09:01 |
| tomwardill | well, forgot | 09:01 |
| tomwardill | trying to do that now | 09:01 |
| cjwatson | On loading arbitrary objects, also note that where launchpadlib normally takes an object as a method parameter, you can in general also pass a URL path. | 09:01 |
| tomwardill | newPushRule appears to be missing from lp.operations on a OCIRecipe | 09:02 |
| tomwardill | oh, is this lp-shell caching WADL? | 09:02 |
| cjwatson | So e.g. instead of foo.bar(distro_series=lp.distributions['ubuntu'].getSeries(name_or_version='focal')) you can say foo.bar(distro_series='/ubuntu/focal') | 09:02 |
| cjwatson | Yes | 09:02 |
| cjwatson | Very annoying and one of these days I'll fix it in launchpadlib | 09:02 |
| tomwardill | how do I fix that again? | 09:02 |
| cjwatson | rm -v ~/.launchpadlib/api.launchpad.net/cache/*.wadl* | 09:03 |
| tomwardill | nope, still only have newWebHook | 09:03 |
| tomwardill | hmm | 09:03 |
| cjwatson | Er sorry, dogfood | 09:04 |
| cjwatson | rm -v ~/.launchpadlib/api.dogfood.paddev.net/cache/*.wadl* | 09:04 |
| tomwardill | ah, of course | 09:04 |
| tomwardill | right, there we go | 09:04 |
| * tomwardill finds his credentials | 09:04 | |
| cjwatson | (The bug is probably in lazr.restfulclient actually) | 09:05 |
| tomwardill | raise RuntimeError("No public key configured")\nRuntimeError: No public key configured\n' | 09:06 |
| cjwatson | You know I absolutely could have sworn I'd done that | 09:09 |
| cjwatson | But no sign of it | 09:09 |
| cjwatson | Please hold | 09:10 |
| cjwatson | tomwardill: Should be fixed | 09:15 |
| tomwardill | well, I've got a push rule | 09:15 |
| tomwardill | starting a new build | 09:16 |
| tomwardill | https://dogfood.paddev.net/~twom/ubuntu/+oci/test-twom-2020-04-22-2/+recipe/test-twom-2020-04-22-2-recipe/+build/8 | 09:16 |
| tomwardill | cjwatson: build done | 09:21 |
| cjwatson | $ /srv/launchpad.net/codelines/current/cronscripts/process-job-source.py -v IOCIRegistryUploadJobSource | 09:23 |
| tomwardill | {"repositories":["busybox","dogfood/test-twom-weget","pjdc/registry-2","registry","test-wget-image","twom/base_with_curl","twom-test/test-wget-image"]} | 09:24 |
| tomwardill | well, there's a thing | 09:24 |
| cjwatson | Seems happy. Would be nice to have some UI indication in LP | 09:24 |
| cjwatson | Also it's super-annoying that lp-shell no longer has good completion after upgrading to focal | 09:26 |
| tomwardill | Status: Downloaded newer image for upload.image-registry.staging.canonical.com:5000/dogfood/test-twom-weget:edge | 09:26 |
| cjwatson | Presumably a Python 3 thing | 09:26 |
| tomwardill | I can docker pull it | 09:26 |
| tomwardill | should probably try it with a more complicated docker image... | 09:26 |
| cjwatson | tomwardill: You can probably move https://trello.com/c/uUeoPcJx/68-ociregistrycredentials-ocipushrule-api to done? | 09:27 |
| tomwardill | done | 09:27 |
| * tomwardill enjoys the confetti | 09:27 | |
| tomwardill | so yeah, we're done right? | 09:29 |
| tomwardill | ;) | 09:29 |
| cjwatson | This does basically look like an MVP: we need https://code.launchpad.net/~cjwatson/lp-production-configs/oci-registry-secrets-public-key/+merge/382733 merged, then a production deployment, then setting up celery and cron for IOCIRegistryUploadJobSource | 09:33 |
| cjwatson | A whole bunch more UI would sure be nice | 09:34 |
| tomwardill | yeah | 09:34 |
| cjwatson | Looks like you can currently create push rules but not list them | 09:35 |
| cjwatson | (on the API) | 09:35 |
| tomwardill | yup | 09:36 |
| tomwardill | I've got a couple of notes of missing things I just found | 09:36 |
| cjwatson | I'm marking your commits green in deployable though since they basically work | 09:36 |
| cjwatson | tomwardill: I'll work on the UI to show the state of registry uploads, if that's OK | 09:52 |
| tomwardill | sure | 09:52 |
| cjwatson | tomwardill: Is the error message handling in OCIRegistryUploadJob.run actually correct for pushing to OCI registries? It seems to be fairly directly taken from snap store interaction, and the "messages" and "details" stuff there is quite specific to the store's API | 10:55 |
| cjwatson | tomwardill: Do you know if the registry API ever returns multiple error messages for a single request? | 10:56 |
| cjwatson | Just wondering if we can simplify this or rationalise it in some way | 10:56 |
| tomwardill | cjwatson: hmm, it may not be. It seemed reasonably flexible for handling possible cases, but I'd need to reread the registry docs for it. I was also working on the assumption that a custom frontend might eventually be a thing | 10:57 |
| tomwardill | https://docs.docker.com/registry/spec/api/#errors-2 the spec is not hugely enlightening | 10:57 |
| cjwatson | tomwardill: The other thing is that in the store case error_message was mostly there for compatibility but you've replicated it; so trying to work out if we can get away with only one of singular vs. plural before fossilising the API | 11:02 |
| tomwardill | cjwatson: yeah, it looks like it | 11:02 |
| cjwatson | tomwardill: errors does generally seem to be possibly-plural there, so I think I'll just consider the plural case in what I'm doing | 14:11 |
| tomwardill | export push_rules on the API: https://code.launchpad.net/~twom/launchpad/+git/launchpad/+merge/382771 | 15:46 |
| cjwatson | r=me | 15:58 |
| tomwardill | ta, landing | 15:59 |
| SpecialK|Canon | cjwatson: sometime between now and Tuesday it would be fab if you could add some status notes to https://trello.com/c/41wQZbTD/26-built-using-support please | 16:01 |
| cjwatson | Yep, plan to | 16:01 |
| cjwatson | Was hoping to get closer to "code all done locally" first | 16:02 |
| cjwatson | tomwardill: did you get a chance to start having a look at built-using-model btw? | 16:02 |
| tomwardill | cjwatson: gah, no, I forgot. Give me link again and I'll do it next? | 16:02 |
| SpecialK|Canon | cjwatson: by all means, cheers | 16:02 |
| cjwatson | tomwardill: https://code.launchpad.net/~cjwatson/launchpad/+git/launchpad/+merge/381234 | 16:03 |
| cjwatson | tomwardill: https://code.launchpad.net/~cjwatson/launchpad/+activereviews and grep for built-using has other useful context | 16:03 |
| cjwatson | (As in, sometimes looking further up the stack may be helpful for understanding | 16:04 |
| cjwatson | ) | 16:04 |
| cjwatson | Also https://docs.google.com/document/d/1HNwxF16tBlh2EjSexihmZj9qP9w5oeGDWF0Fygb8ols is the hopefully-not-too-bitrotten design doc | 16:04 |
| tomwardill | i need to find a more effective way to review things, I really struggle with it just scrolling in a browser | 16:43 |
| cjwatson | I sometimes add a remote and fetch locally | 16:45 |
| tomwardill | cjwatson: it all looks reasonable to me, I'll go over it again in the morning, but I think I can follow it :) | 17:06 |
| cjwatson | Finally landing removal of --system-site-packages now that it's unblocked | 21:39 |
| * cjwatson looks forward to fewer incomprehensible pip bugs | 21:39 | |
| * pappacena \o/ | 21:39 | |
| cjwatson | I did actually have something else blocked on that for some reason I never quite tracked down - a ZTK upgrade I think | 21:40 |
| cjwatson | Something to do with ampoule child processes preferring the system zope.interface to the virtualenv's one because why | 21:41 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!