/srv/irclogs.ubuntu.com/2022/02/03/#cloud-init.txt

falcojr"tox -e integration-tests" would be the best way to run integration tests02:46
falcojryou can pass pytest arguments to it02:46
falcojre.g., tox -e integration-tests tests/integration_tests/modules/test_combined.py 02:46
falcojrI'll update the documentation accordingly02:47
slingamnoh cool03:13
meenaholmanb: any progress on that patch? need help? a review? ping me on github as @igalic14:35
holmanbmeena: apologies, I misunderstood. I thought you were going to submit a PR. I'll ping for a review on github shortly14:47
meenaholmanb: I am, sadly, still swamped in work and childcare duties14:48
holmanbmeena: no worries :)14:48
meenatrying to open my laptop while my toddler is awake, means someone is climbing on me, and that doesn't seem conducive to … doing things14:49
meenalike, the patch is right there! emaste wrote it :P somebody who's signed a CLA just needs to submit it, and then somebody will ask, but what about tests??!!?14:49
holmanbmeena for review when you get a chance: https://github.com/canonical/cloud-init/pull/1236/files14:54
ubottuPull 1236 in canonical/cloud-init "cc_salt_minion freebsd fix for rc.conf" [Open]14:54
meenaholmanb: that'll now break on linux. You want this patch: https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=254339#c514:56
ubottubugs.freebsd.org bug 254339 in Ports & Packages "net/cloud-init: fix modifying rc.conf in cc_salt_minion" [Affects Only Me, Open]14:56
meenano, never mind14:57
meenathere is no linux code in there lol14:57
meenaor rather, it's done differently somehow, or it's not necessary14:58
meenaand on line 64, we could also rename that package to: 'sysutils/py-salt' that would make it future proof. cuz there's no py36-salt anymore15:00
holmanbmeena: +1 updated15:03
meenaholmanb: and tests still somehow pass?15:04
holmanbmeena: checking now15:06
meenaoh, we're still using Travis CI…15:07
holmanbmeena: unit and integration tests passed, I just pushed a small format change15:22
meenahrm, forgot that my review isn't worth a green tick15:23
holmanbmeena: Your review is still appreciated, thanks!15:25
=== djhankb6 is now known as djhankb
blackboxswmeena: holmanb thanks. +1 on that PR. do you think it's worth calling out Co-Authored-By:  b..@...bsd.org in the changelog there?19:44
blackboxswor do we/BSD contributor emails prefer to be anonymous upstream?19:46
blackboxswstrange question of attribution in this case for me.19:48
holmanb@blackboxsw: +1 good call asking, I'm definitely not the original author :-)19:50
blackboxswfor sure I'm sitting at the "Squash merge" button in github, so figured we should make sure context on that commit is desired.  Also, for BSD cases where we have bugzilla references I wanted to introduce a "bsdbz: #<bug_id>" footer like we have done w/ redhat via "rhbz: <id>|url" and "boo: <id>" 19:53
blackboxswboo === suse like https://bugzilla.suse.com/show_bug.cgi?id=1080595  not sure why that abbreviation though19:54
ubottubugzilla.suse.com bug 1080595 in openSUSE Distribution "cloud-init uses missing wheel group" [Normal, Resolved: Fixed]19:54
holmanbhmmm, nothing about boo implies suse to my brain, but bugzilla.opensuse.org -> b.o.o perhaps?19:57
blackboxswahhh yes b.o.[o]rg20:04
falcojrblackboxsw and paride : now that I think about it, I'm not sure we should have jenkins jobs that use IN_PLACE . If we want things to be faster, we can still git clone the repo and run the tests out of git, but instead just specify ppa:cloud-init-dev/daily as the CLOUD_INIT_SOURCE21:50
blackboxswfalcojr: only diff we have there though is that it's a two hour turnaround after something commits to trunk that optimally we can see those packages hit daily.21:51
falcojrbut as pointed out in https://github.com/canonical/cloud-init/pull/1237#pullrequestreview-872443537 , we'll have failures due to configuration files not being in place, and they'll only be in the right place if we use an actual package21:52
ubottuPull 1237 in canonical/cloud-init "testing: stop universally overwriting /etc/cloud/cloud.cfg.d" [Open]21:52
blackboxswthat said. grr. ubuntu/bionic daily is recipe broken too. I think when we landed the new-upstream-snapshot to handle black formatting, we inadvertently introduced merge conflicts again main.21:53
falcojrwe need notifications for when daily builds fail21:54
falcojrblackboxsw: this doesn't appear to be black related21:59
falcojrit's yelling about some of the recent schema files which are post-black...not sure why we're not seeing a clean upstream snapshot21:59
blackboxswyeah, though merge conflicts  started Jan 24th I think21:59
blackboxswjust starting to look too. it coincides with commit 872eebd2cbe6898260e4a8559ce07dc452d84ff422:00
blackboxswif I git reset to that commit and try new-upstream-snapshot things succeed without conflict I think. double checking22:01
falcojrOH...did we squash that?22:01
falcojrthe upstream snapshot is a single commit22:01
falcojrwe'll have to reset before that and upstream snapshot again22:02
blackboxswyep. darn it. you're right. bbsw squash merge button blame :/22:05
falcojrblackboxsw: Haha...all good. I've almost done that on packaging branches too many times22:06
blackboxswyep will dr the branch now.22:06
falcojrblackboxsw: actually, we could probably interactively rebase and just remove that commit altogether22:08
falcojrthen do a new-upstream-snapshot22:08
falcojrmight be the easiest way to keep our packaging commits22:08
falcojrahh, wait...nm22:08
falcojrthere's packaging changes in there too22:09
blackboxswyeah it's a bit more annoying :/22:09
blackboxswfalcojr, holmanb: my blackboxsw/ubuntu/bionic is the unsquashed version of those commits that I rebased/replayed from brett's PR 1203 and JAmes' 122022:19
falcojrblackboxsw: did you close the PR already? And how'd you do that so quickly?22:19
blackboxswthere's no git diff in the upstream/ubuntu/bionic and blackboxsw/ubuntu/bionic but the commit history is different22:20
blackboxswfalcojr: I closed the PR because I accidentally targeted it to upstream/main22:20
blackboxswbut github doesn't allow me to create a PR against ubuntu/main because there is no "diff" to speak of (only git history diffs)22:20
blackboxswI mean I can't create a PR from blackboxsw/ubuntu/boinic which targets  upstream/ubuntu/bionic22:21
falcojrhere's my +1 if there's no diff...we usually merge those on the CLI anyway22:21
blackboxswthe only way to "fix" my error here is either --force push to correct git history there (which should not break our build recipes because they base on number of commits past last annotated upstream version)22:22
blackboxswyeah will force push that now and kickoff bionic build recipe22:22
blackboxswthx22:22
blackboxswand yes we need a better signal for recipe failures22:22
holmanbblackboxsw/ubuntu/bionic doesn't inlcude 1203 or 1220, looks like the latest hash is 65bba522:28
holmanbam I looking at the wrong thing?22:28
blackboxswholmanb: you are looking at the right thing. /me forgot to hit enter on my git push --force22:30
blackboxswholmanb: commit should be a00215bb2139b26859df21121136e6b731132e5a22:31
holmanbI see it now, thx22:31
falcojrblackboxsw: is that local, or did you push that to origin?22:31
holmanbfalcojr: https://github.com/blackboxsw/cloud-init/commits/ubuntu/bionic22:32
blackboxswfalcojr: it is "local to blackboxsw remote" I haven't pushed to origin yet. And github does let me create a PR (now that I correctly pushed to my repo)22:32
falcojrnm...I had seen the same thing as Brett, but after a fetch they match now22:32
blackboxswI'm putting a PR up now that github can see the difference so we can track this22:35
blackboxswholmanb:  falcojr https://github.com/canonical/cloud-init/pull/124222:42
ubottuPull 1242 in canonical/cloud-init "Ubuntu/bionic" [Open]22:42
blackboxswif approved. I'll force push this to ubuntu/bionic (once I assert merging and sbuild generates the package without errors22:42
blackboxswforced update to make sure I'm using the specific cherry-pick from tip of upsream/ubuntu/bionic stlll no diff, but proper authorship etc.22:49
blackboxswrunning through sbuild etc now and will post results to PR.22:50
blackboxswthanks folks. build package is good. just walking throgh manual recipe merge steps to confirm main merge ubuntu/bionic without conflicts23:00
blackboxswcode sync'd for bionic via jenkins jobs. I'm kicking off the bionic build recipe and will kick off subsequent jenkins jobs for bionic once daily PPA contains latest 23:14

Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!