opendevreview | Ian Wienand proposed opendev/base-jobs master: DNM: trying to fail job https://review.opendev.org/c/opendev/base-jobs/+/844754 | 00:52 |
---|---|---|
ianw | that df thing has got me a few times too. interestingly it worked for ^ | 00:59 |
ianw | i grepped all the executor logs, and didn't find any current instances where it seemed to fail | 01:03 |
opendevreview | Ian Wienand proposed opendev/base-jobs master: DNM: trying to fail job https://review.opendev.org/c/opendev/base-jobs/+/844754 | 01:12 |
opendevreview | Ian Wienand proposed opendev/base-jobs master: DNM: trying to fail job https://review.opendev.org/c/opendev/base-jobs/+/844754 | 01:39 |
opendevreview | Ian Wienand proposed opendev/base-jobs master: DNM: trying to fail job https://review.opendev.org/c/opendev/base-jobs/+/844754 | 03:25 |
ianw | interesting, timing out the job and also filling up the disk during execution don't appear to trigger this behaviour | 03:49 |
opendevreview | Ian Wienand proposed opendev/base-jobs master: DNM: trying to fail job https://review.opendev.org/c/opendev/base-jobs/+/844754 | 04:01 |
*** gibi is now known as gibi_summit | 06:01 | |
StutiArya[m] | Hi, I am trying to push my stable branch changes to master branch, whenever I try to push I get below error :... (full message at https://matrix.org/_matrix/media/r0/download/matrix.org/JjcFancgqWYvHlvuLFZqMCrF) | 07:20 |
Clark[m] | Stuti Arya: can you share what repository you are working with and what command you are running? | 07:21 |
fungi | fwiw, that doesn't look like an error from git-review itself | 07:25 |
StutiArya[m] | I have made changes to stable/xena branch in x/networking-infoblox, then did... (full message at https://matrix.org/_matrix/media/r0/download/matrix.org/tntgCQhDTjFNdbuNVFnTGmcX) | 07:26 |
fungi | StutiArya[m]: make sure you're following https://docs.opendev.org/opendev/infra-manual/latest/gettingstarted.html#getting-started for setting up your account and tools correctly | 07:28 |
StutiArya[m] | fungi: yes, I am following this Document | 07:29 |
Clark[m] | Stuti Arya: the issue is that `git push` takes a remote as the first argument. In this case there is no git remote named master. But generally our docs won't recommend pushing directly | 07:30 |
Clark[m] | the expectation is that all changes are pushed for review to special gerrit refs (something that git-review does for you automatically) and not directly to the branch | 07:30 |
fungi | https://docs.opendev.org/opendev/infra-manual/latest/gettingstarted.html#proposing-a-change | 07:30 |
fungi | (note nowhere does it tell you to use `git push`) | 07:31 |
fungi | StutiArya[m]: can you be more specific about what you mean by "push my stable branch changes to master branch"? are you using git cherry-pick to pick those changes to the other branch? | 07:33 |
fungi | the exact commands you're running and the repository you're running them on will be necessary for us to be able to provide better guidance | 07:33 |
Clark[m] | fungi: the steps are in one of the previous messages which may have been truncated by irc with a link to them | 07:33 |
fungi | oh, i missed the second message, yes | 07:34 |
StutiArya[m] | fungi: I am running below cmds:... (full message at https://matrix.org/_matrix/media/r0/download/matrix.org/QBCeKGhBVzeQijRAWjsPJOYZ) | 07:34 |
fungi | okay, so you're trying to push a merge commit, it sounds like? | 07:35 |
*** ysandeep is now known as ysandeep|lunch | 07:36 | |
StutiArya[m] | > <@sarya009:matrix.org> fungi: I am running below cmds:... (full message at https://matrix.org/_matrix/media/r0/download/matrix.org/QsYWMUnwftYNVDrNTLzGVECd) | 07:36 |
fungi | anyway, `git review master` will override the branch detection from the stable branch .gitreview file, but you're probably better off cherry-picking those commits to master and then `git review` in master should pick the correct branch and create new changes | 07:36 |
fungi | also, your `git merge stable` is overwriting the master branch .gitreview file, which is a bad idea | 07:39 |
fungi | those files need to be branch-specific | 07:39 |
clarkb | looking at the POST_FAILURES monasca-common fails due to the git cross user permissions error in a post run playbook. I think we can ignore that set of failures | 07:43 |
clarkb | kolla has an error with a read timeout pushing to quay.io. Can also ignor ethat one | 07:44 |
clarkb | glance failed devstack which prevented glance from being installed which broke some post-run play that runs glance commands | 07:45 |
clarkb | the more I look at not OSA the more I think the OSA problems may be OSA specific | 07:45 |
fungi | seems that way, yes | 07:46 |
fungi | lemme see if the autohold from last night caught anything yet | 07:46 |
fungi | yeah, 104.239.175.148 looks like one. that was for node request 300-0018332716 associated with build https://zuul.opendev.org/t/openstack/build/4b0e0c1d64c0471f8b1ac330d21b6e22 which was a stable/wallaby periodic pipeline job which ended in a normal FAILURE result due to what looks like it broke when trying to start some nova services | 07:56 |
fungi | noonedeadpunk: i'm guessing that's not useful, so i'm going to release it and reset the autohold | 07:56 |
fungi | if you want it scoped to a specific change you're rechecking in order to try to reproduce the problem, that may help narrow it a bit | 07:56 |
fungi | anyway, i've got a general autohold for --project=openstack/openstack-ansible --tenant=openstack --job=openstack-ansible-deploy-aio_lxc-ubuntu-focal at the moment | 07:58 |
clarkb | https://github.com/jwilk/python-syntax-errors is an interesting idea | 08:03 |
fungi | i agree, it's an awesome idea... but no license declared for any of it? | 08:15 |
fungi | i would feel weird reusing those | 08:15 |
StutiArya[m] | <fungi> "anyway, `git review master` will..." <- is there any other method than cherry pick? | 08:19 |
fungi | StutiArya[m]: updating your project's gerrit acl to allow pushing merge commits | 08:20 |
fungi | but those are very hard to review and can create a mess if you're not extremely careful | 08:21 |
StutiArya[m] | > <@_oftc_fungi:matrix.org> anyway, `git review master` will override the branch detection from the stable branch .gitreview file, but you're probably better off cherry-picking those commits to master and then `git review` in master should pick the correct branch and create new changes | 08:21 |
StutiArya[m] | * is there any other method than cherry pick? | 08:21 |
StutiArya[m] | I am getting version compatibility error with master, but originally stable/xena was created from master | 08:21 |
fungi | for example, messes like just merging to master without being careful to omit the .gitreview file | 08:21 |
fungi | i don't understand what "getting version compatibility error with master" means | 08:22 |
fungi | please reference specific error messages and specific commands which are resulting in them | 08:23 |
StutiArya[m] | https://review.opendev.org/c/x/networking-infoblox/+/843498 | 08:23 |
fungi | "ERROR: Cannot install Babel==2.9.1 because these package versions have conflicting dependencies" https://zuul.opendev.org/t/openstack/build/489d7839fc8f4d95904eab1eb1d1c340/log/job-output.txt#636 | 08:26 |
fungi | that looks like a python packaging version mismatch | 08:26 |
StutiArya[m] | why this is not working with master branch and working with stable branch | 08:27 |
StutiArya[m] | if I will go and change stable/branch maybe I will get new version mismatch errors | 08:27 |
StutiArya[m] | Is there any solution for this? | 08:28 |
fungi | you could try to reproduce it locally and compare `tox -e pep8` output between stable/xena and master | 08:30 |
StutiArya[m] | fungi: https://zuul.opendev.org/t/openstack/build/0d93b1396ced40ab8634dba987ef1087 | 08:44 |
StutiArya[m] | * fungi: https://zuul.opendev.org/t/openstack/build/0d93b1396ced40ab8634dba987ef1087 | 08:45 |
StutiArya[m] | I am getting this error now- ERROR: Cannot install Babel==2.10.1 because these package versions have conflicting dependencies | 08:45 |
fungi | pep8 installdeps: -c/home/zuul/src/opendev.org/openstack/requirements/upper-constraints.txt, -r/home/zuul/src/opendev.org/x/networking-infoblox/requirements.txt, -r/home/zuul/src/opendev.org/x/networking-infoblox/test-requirements.txt | 08:48 |
fungi | https://opendev.org/openstack/requirements/src/branch/master/upper-constraints.txt#L559 | 08:49 |
StutiArya[m] | what rebase does ? | 08:52 |
fungi | oh, this is on stable/xena not master | 08:53 |
StutiArya[m] | fungi: yes | 08:53 |
fungi | https://opendev.org/x/networking-infoblox/src/branch/stable/xena/requirements.txt#L5 and https://opendev.org/openstack/requirements/src/branch/stable/xena/upper-constraints.txt#L559 disagree, which is the reason for your error | 08:55 |
fungi | this has nothing to do with opendev, this is an openstack-related software development problem you're having | 08:56 |
StutiArya[m] | How can I resolve this? | 09:01 |
fungi | avoid using specific version pins in your requirements files if you're also going to use pinned constraints | 09:04 |
fungi | your master branch does "Babel>=1.3" which is fine in combination with the "Babel===2.10.1" constraint in the openstack/requirements master branch | 09:06 |
fungi | but on stable/xena you have "Babel==2.9.1" which conflicts with the "Babel===2.9.1" constraint in the stable/xena branch of openstack/requirements | 09:07 |
fungi | er, no that's the same version isn't it | 09:08 |
* fungi looks closer | 09:08 | |
clarkb | it might be a python version problem | 09:08 |
clarkb | looks like maybe 3.6 or newer is required | 09:09 |
fungi | oh, that's coming from your proposed change: https://review.opendev.org/c/x/networking-infoblox/+/844789/1/requirements.txt | 09:09 |
clarkb | oh thats for latest babel | 09:09 |
clarkb | 2.9.1 does python2.7 and 3.4 or newer | 09:09 |
fungi | so the error message is quite straightforward. your change updates requirements.txt to "Babel==2.10.1" which conflicts with the "Babel===2.9.1" in stable/xena of openstack/requirements | 09:10 |
*** ysandeep|lunch is now known as ysandeep | 09:15 | |
*** jpena|off is now known as jpena | 09:37 | |
opendevreview | Clark Boylan proposed zuul/zuul-jobs master: Add thhe post-reboot-tasks role https://review.opendev.org/c/zuul/zuul-jobs/+/844704 | 09:41 |
opendevreview | Clark Boylan proposed zuul/zuul-jobs master: Add the post-reboot-tasks role https://review.opendev.org/c/zuul/zuul-jobs/+/844704 | 10:03 |
clarkb | I'm not sure why the linter is failing on that now. It seems like it is angry about things in othe rfiles which might indicate a new version of the linter however I recognize some of those rules as being old ? I dunno | 10:04 |
clarkb | we skip experimental so maybe those rules moved from experimental to not experimental | 10:08 |
opendevreview | Clark Boylan proposed zuul/zuul-jobs master: Add the post-reboot-tasks role https://review.opendev.org/c/zuul/zuul-jobs/+/844704 | 10:13 |
clarkb | if you look at the diff between those two patchsets please don't laugh at me. I blame jet la | 10:15 |
*** pojadhav is now known as pojadhav|afk | 11:14 | |
*** ysandeep is now known as ysandeep|break | 11:39 | |
opendevreview | Clark Boylan proposed zuul/zuul-jobs master: Add the post-reboot-tasks role https://review.opendev.org/c/zuul/zuul-jobs/+/844704 | 11:45 |
*** dviroel|out is now known as dviroel | 11:51 | |
opendevreview | Clark Boylan proposed zuul/zuul-jobs master: Add the post-reboot-tasks role https://review.opendev.org/c/zuul/zuul-jobs/+/844704 | 11:56 |
*** ysandeep|break is now known as ysandeep | 12:46 | |
*** pojadhav|afk is now known as pojadhav | 13:18 | |
*** ysandeep is now known as ysandeep|brb | 14:25 | |
*** ysandeep|brb is now known as ysandeep | 14:33 | |
*** dviroel is now known as dviroel|lunch | 15:12 | |
johnsom | Is there a way that I am missing for a core to remove the "Work In Progress" flag in gerrit that someone else pressed? Not the "-1" WIP vote, but the new "Mark as work in progress" option? Someone set this on a patch because another patch needed to land first (yes, I know) then went on vacation. This seems to be blocking the +W on that patch. | 15:20 |
johnsom | https://review.opendev.org/c/openstack/designate/+/844152 | 15:20 |
johnsom | Updates to the patch don't seem to clear it either. | 15:21 |
Clark[m] | johnsom: not currently. That is the main reason we haven't dropped the -W vote system | 15:27 |
Clark[m] | I think an admin can unset it but I'm not sure of the details on that | 15:27 |
johnsom | Ok, thanks. I don' think this is critical, but was wondering if I was missing something. | 15:28 |
fungi | yeah, right now there's no acl to control it, so it's only possible for the change owner or admins to set/remove | 15:43 |
*** marios is now known as marios|out | 15:45 | |
*** ysandeep is now known as ysandeep|out | 15:53 | |
*** dviroel|lunch is now known as dviroel | 16:22 | |
*** jpena is now known as jpena|off | 17:15 | |
opendevreview | Merged openstack/project-config master: grafana ceph: add nova stable/(xena|yoga) branches https://review.opendev.org/c/openstack/project-config/+/835514 | 18:54 |
*** mnasiadka_ is now known as mnasiadka | 19:53 | |
*** dviroel is now known as dviroel|afk | 19:57 | |
code_bleu | hello | 21:22 |
code_bleu | well poo.. I guess this client isn't working...no one saying anything. if you do see this, I'd appreciate the acknowledgement that it works...and won't bug you anymore. | 21:25 |
BlaisePabon[m] | LGTM | 21:26 |
BlaisePabon[m] | code_bleu: Di you have a specific question? | 21:27 |
code_bleu | BlaisePabon[m] thanks...now I know it works at least | 21:27 |
code_bleu | yes | 21:27 |
code_bleu | I've been struggling to find the right path to get policy.yaml file to work. I have the config in nova.conf setup with policy_dirs and policy_file...I used the oslopolicy-sample-generator --namespace nova to create the policy.yaml file. For testing, I'm trying to get a non-admin user be able to do an 'openstack host list', but nothing is working | 21:27 |
fungi | code_bleu: that sounds like an openstack project question. the #opendev channel is where we discuss our source code hosting, collaboration and test automation platform, per the /topic | 22:09 |
fungi | you probably want the #openstack channel, or to post on the openstack-discuss@lists.openstack.org mailing list | 22:10 |
opendevreview | Merged openstack/diskimage-builder master: Remove py 3.6 support and update jobs https://review.opendev.org/c/openstack/diskimage-builder/+/836229 | 23:18 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!