)]}'
{"id":"openstack%2Fironic-python-agent~656275","triplet_id":"openstack%2Fironic-python-agent~master~I63a6ff91d702b990ef09da13e6e2e269e7274cce","project":"openstack/ironic-python-agent","branch":"master","topic":"bug/2005528","hashtags":[],"change_id":"I63a6ff91d702b990ef09da13e6e2e269e7274cce","subject":"Add more channel number for detecting BMC IP address","status":"MERGED","created":"2019-04-29 09:33:16.000000000","updated":"2019-05-08 02:06:49.000000000","submitted":"2019-05-08 02:03:44.000000000","submitter":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"total_comment_count":1,"unresolved_comment_count":0,"has_review_started":true,"submission_id":"656275-1557281024961-a11cbdcb","meta_rev_id":"2e4d0c0d222a07c2b4605b8bea76549438a1ed97","_number":656275,"virtual_id_number":656275,"owner":{"_account_id":12860,"name":"Dongcan Ye","email":"yedongcan@yeah.net","username":"yedongcan"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"value":0,"_account_id":11076,"name":"Shivanand Tendulker","email":"stendulker@gmail.com","username":"stendulker"},{"value":0,"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},{"value":0,"_account_id":10343,"name":"Jim Rollenhagen","email":"jim@jimrollenhagen.com","username":"jimrollenhagen"},{"value":0,"_account_id":13689,"name":"Hironori Shiina","email":"Hironori.Shiina@fujitsu.com","username":"shiina"},{"value":0,"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},{"value":0,"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},{"value":0,"_account_id":23851,"name":"Riccardo Pittau","email":"elfosardo@gmail.com","username":"elfosardo"},{"value":0,"_account_id":11292,"name":"Arne Wiebalck","email":"Arne.Wiebalck@cern.ch","username":"wiebalck"},{"value":0,"_account_id":24828,"name":"Kaifeng Wang","email":"kaifeng.w@gmail.com","username":"wangkf"},{"value":2,"date":"2019-05-08 02:03:44.000000000","post_submit":true,"permitted_voting_range":{"min":2,"max":2},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"_account_id":12860,"name":"Dongcan Ye","email":"yedongcan@yeah.net","username":"yedongcan"},{"value":0,"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"}],"values":{"-2":"Fails","-1":"Doesn\u0027t seem to work"," 0":"No score","+1":"Works for me","+2":"Verified"},"description":"","default_value":0,"optional":true},"Code-Review":{"approved":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"all":[{"value":0,"_account_id":11076,"name":"Shivanand Tendulker","email":"stendulker@gmail.com","username":"stendulker"},{"value":0,"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},{"value":0,"_account_id":10343,"name":"Jim Rollenhagen","email":"jim@jimrollenhagen.com","username":"jimrollenhagen"},{"value":0,"_account_id":13689,"name":"Hironori Shiina","email":"Hironori.Shiina@fujitsu.com","username":"shiina"},{"value":0,"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},{"value":2,"date":"2019-05-07 15:43:02.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},{"value":0,"_account_id":23851,"name":"Riccardo Pittau","email":"elfosardo@gmail.com","username":"elfosardo"},{"value":0,"_account_id":11292,"name":"Arne Wiebalck","email":"Arne.Wiebalck@cern.ch","username":"wiebalck"},{"value":2,"date":"2019-05-08 01:01:01.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":24828,"name":"Kaifeng Wang","email":"kaifeng.w@gmail.com","username":"wangkf"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"_account_id":12860,"name":"Dongcan Ye","email":"yedongcan@yeah.net","username":"yedongcan"},{"value":0,"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"}],"values":{"-2":"Do not merge","-1":"This patch needs further work before it can be merged"," 0":"No score","+1":"Looks good to me, but someone else must approve","+2":"Looks good to me (core reviewer)"},"description":"","default_value":0,"optional":true},"Workflow":{"approved":{"_account_id":24828,"name":"Kaifeng Wang","email":"kaifeng.w@gmail.com","username":"wangkf"},"all":[{"value":0,"date":"2019-05-05 10:31:37.000000000","_account_id":11076,"name":"Shivanand Tendulker","email":"stendulker@gmail.com","username":"stendulker"},{"value":0,"date":"2019-05-05 10:31:37.000000000","permitted_voting_range":{"min":0,"max":1},"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},{"value":0,"date":"2019-05-05 10:31:37.000000000","_account_id":10343,"name":"Jim Rollenhagen","email":"jim@jimrollenhagen.com","username":"jimrollenhagen"},{"value":0,"date":"2019-05-05 10:31:37.000000000","_account_id":13689,"name":"Hironori Shiina","email":"Hironori.Shiina@fujitsu.com","username":"shiina"},{"value":0,"date":"2019-05-05 10:31:37.000000000","_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},{"value":0,"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},{"value":0,"_account_id":23851,"name":"Riccardo Pittau","email":"elfosardo@gmail.com","username":"elfosardo"},{"value":0,"date":"2019-05-07 16:07:26.000000000","_account_id":11292,"name":"Arne Wiebalck","email":"Arne.Wiebalck@cern.ch","username":"wiebalck"},{"value":1,"date":"2019-05-08 01:01:01.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":24828,"name":"Kaifeng Wang","email":"kaifeng.w@gmail.com","username":"wangkf"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"_account_id":12860,"name":"Dongcan Ye","email":"yedongcan@yeah.net","username":"yedongcan"},{"value":0,"date":"2019-05-05 10:31:37.000000000","permitted_voting_range":{"min":0,"max":1},"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Backport-Candidate":{"all":[{"value":0,"_account_id":11076,"name":"Shivanand Tendulker","email":"stendulker@gmail.com","username":"stendulker"},{"value":0,"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},{"value":0,"_account_id":10343,"name":"Jim Rollenhagen","email":"jim@jimrollenhagen.com","username":"jimrollenhagen"},{"value":0,"_account_id":13689,"name":"Hironori Shiina","email":"Hironori.Shiina@fujitsu.com","username":"shiina"},{"value":0,"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},{"value":0,"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},{"value":0,"_account_id":23851,"name":"Riccardo Pittau","email":"elfosardo@gmail.com","username":"elfosardo"},{"value":0,"_account_id":11292,"name":"Arne Wiebalck","email":"Arne.Wiebalck@cern.ch","username":"wiebalck"},{"value":0,"_account_id":24828,"name":"Kaifeng Wang","email":"kaifeng.w@gmail.com","username":"wangkf"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"_account_id":12860,"name":"Dongcan Ye","email":"yedongcan@yeah.net","username":"yedongcan"},{"value":0,"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"}],"values":{"-1":"Do Not Backport"," 0":"Backport Review Needed","+1":"Should Backport"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},{"_account_id":10343,"name":"Jim Rollenhagen","email":"jim@jimrollenhagen.com","username":"jimrollenhagen"},{"_account_id":11076,"name":"Shivanand Tendulker","email":"stendulker@gmail.com","username":"stendulker"},{"_account_id":11292,"name":"Arne Wiebalck","email":"Arne.Wiebalck@cern.ch","username":"wiebalck"},{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},{"_account_id":12860,"name":"Dongcan Ye","email":"yedongcan@yeah.net","username":"yedongcan"},{"_account_id":13689,"name":"Hironori Shiina","email":"Hironori.Shiina@fujitsu.com","username":"shiina"},{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"_account_id":23851,"name":"Riccardo Pittau","email":"elfosardo@gmail.com","username":"elfosardo"},{"_account_id":24828,"name":"Kaifeng Wang","email":"kaifeng.w@gmail.com","username":"wangkf"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2019-05-01 09:43:00.000000000","updated_by":{"_account_id":23851,"name":"Riccardo Pittau","email":"elfosardo@gmail.com","username":"elfosardo"},"reviewer":{"_account_id":23851,"name":"Riccardo Pittau","email":"elfosardo@gmail.com","username":"elfosardo"},"state":"REVIEWER"},{"updated":"2019-05-05 10:31:37.000000000","updated_by":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"reviewer":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"state":"REVIEWER"},{"updated":"2019-05-05 10:31:37.000000000","updated_by":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"reviewer":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"state":"REVIEWER"},{"updated":"2019-05-05 10:31:37.000000000","updated_by":{"_account_id":10343,"name":"Jim Rollenhagen","email":"jim@jimrollenhagen.com","username":"jimrollenhagen"},"reviewer":{"_account_id":10343,"name":"Jim Rollenhagen","email":"jim@jimrollenhagen.com","username":"jimrollenhagen"},"state":"REVIEWER"},{"updated":"2019-05-05 10:31:37.000000000","updated_by":{"_account_id":11076,"name":"Shivanand Tendulker","email":"stendulker@gmail.com","username":"stendulker"},"reviewer":{"_account_id":11076,"name":"Shivanand Tendulker","email":"stendulker@gmail.com","username":"stendulker"},"state":"REVIEWER"},{"updated":"2019-05-05 10:31:37.000000000","updated_by":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"reviewer":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"state":"REVIEWER"},{"updated":"2019-05-05 10:31:37.000000000","updated_by":{"_account_id":13689,"name":"Hironori Shiina","email":"Hironori.Shiina@fujitsu.com","username":"shiina"},"reviewer":{"_account_id":13689,"name":"Hironori Shiina","email":"Hironori.Shiina@fujitsu.com","username":"shiina"},"state":"REVIEWER"},{"updated":"2019-05-07 15:43:02.000000000","updated_by":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"reviewer":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"state":"REVIEWER"},{"updated":"2019-05-07 16:07:26.000000000","updated_by":{"_account_id":11292,"name":"Arne Wiebalck","email":"Arne.Wiebalck@cern.ch","username":"wiebalck"},"reviewer":{"_account_id":11292,"name":"Arne Wiebalck","email":"Arne.Wiebalck@cern.ch","username":"wiebalck"},"state":"REVIEWER"},{"updated":"2019-05-08 01:01:01.000000000","updated_by":{"_account_id":24828,"name":"Kaifeng Wang","email":"kaifeng.w@gmail.com","username":"wangkf"},"reviewer":{"_account_id":24828,"name":"Kaifeng Wang","email":"kaifeng.w@gmail.com","username":"wangkf"},"state":"REVIEWER"},{"updated":"2019-05-08 02:03:44.000000000","updated_by":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"reviewer":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"state":"REVIEWER"}],"messages":[{"id":"dc471a848e92bd9491970ab0475b791eb736d455","author":{"_account_id":12860,"name":"Dongcan Ye","email":"yedongcan@yeah.net","username":"yedongcan"},"date":"2019-04-29 09:33:16.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"25371b74df0a2032763700903403621ddb9f90e8","author":{"_account_id":12860,"name":"Dongcan Ye","email":"yedongcan@yeah.net","username":"yedongcan"},"date":"2019-04-29 09:35:41.000000000","message":"Uploaded patch set 2: Commit message was updated.","accounts_in_message":[],"_revision_number":2},{"id":"26c1a731edd05528c5d326b63c507b5bd722c5a5","author":{"_account_id":23851,"name":"Riccardo Pittau","email":"elfosardo@gmail.com","username":"elfosardo"},"date":"2019-04-29 09:54:41.000000000","message":"Patch Set 2:\n\nHi Dongcan, which version/revision of IPMI are you referring to?\nCould you please add the doc url reference of the IPMI specification?\nThank you!","accounts_in_message":[],"_revision_number":2},{"id":"8754f239d4ee0d02943c2cdb057b317ba12ee830","author":{"_account_id":12860,"name":"Dongcan Ye","email":"yedongcan@yeah.net","username":"yedongcan"},"date":"2019-04-29 10:28:41.000000000","message":"Patch Set 2:\n\n\u003e Hi Dongcan, which version/revision of IPMI are you referring to?\n \u003e Could you please add the doc url reference of the IPMI\n \u003e specification?\n \u003e Thank you!\n\nHi, my IPMI version is 2.0. And from the docs[1] we can lookup chapter 6.2 \"Channel Numbers\". The channel number 1-Bh means 1-11.\n\n[1] https://www.intel.com/content/dam/www/public/us/en/documents/product-briefs/ipmi-second-gen-interface-spec-v2-rev1-1.pdf","accounts_in_message":[],"_revision_number":2},{"id":"28ab28a15ebba5bd7eef26b854adff33f7642aec","author":{"_account_id":12860,"name":"Dongcan Ye","email":"yedongcan@yeah.net","username":"yedongcan"},"date":"2019-04-29 10:30:00.000000000","message":"Patch Set 3: Commit message was updated.","accounts_in_message":[],"_revision_number":3},{"id":"20717564b7fa31b0dd1edbc526c7398f49da7320","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-04-29 12:09:24.000000000","message":"Patch Set 3: Verified+1\n\nBuild succeeded (check pipeline).\n\n- openstack-tox-pep8 http://logs.openstack.org/75/656275/3/check/openstack-tox-pep8/2d885f6/ : SUCCESS in 2m 41s\n- openstack-tox-py27 http://logs.openstack.org/75/656275/3/check/openstack-tox-py27/1026465/ : SUCCESS in 3m 08s\n- openstack-tox-py36 http://logs.openstack.org/75/656275/3/check/openstack-tox-py36/a89097f/ : SUCCESS in 2m 55s\n- openstack-tox-cover http://logs.openstack.org/75/656275/3/check/openstack-tox-cover/3d1f9e7/cover/ : SUCCESS in 3m 09s\n- openstack-tox-docs http://logs.openstack.org/75/656275/3/check/openstack-tox-docs/8f92a57/html/ : SUCCESS in 3m 26s\n- ipa-tempest-partition-bios-ipmi-direct-tinyipa-src http://logs.openstack.org/75/656275/3/check/ipa-tempest-partition-bios-ipmi-direct-tinyipa-src/826ad65/ : SUCCESS in 1h 10m 31s\n- ipa-tempest-partition-bios-ipmi-iscsi-tinyipa-src http://logs.openstack.org/75/656275/3/check/ipa-tempest-partition-bios-ipmi-iscsi-tinyipa-src/2e744a7/ : SUCCESS in 57m 23s\n- ipa-tempest-wholedisk-bios-ipmi-direct-tinyipa-src http://logs.openstack.org/75/656275/3/check/ipa-tempest-wholedisk-bios-ipmi-direct-tinyipa-src/066b496/ : SUCCESS in 1h 22m 28s\n- ipa-tempest-wholedisk-bios-ipmi-iscsi-tinyipa-src http://logs.openstack.org/75/656275/3/check/ipa-tempest-wholedisk-bios-ipmi-iscsi-tinyipa-src/28425c0/ : SUCCESS in 1h 08m 01s\n- ipa-tempest-ironic-inspector-src http://logs.openstack.org/75/656275/3/check/ipa-tempest-ironic-inspector-src/105816a/ : SUCCESS in 1h 38m 19s (non-voting)\n- ipa-tempest-dsvm-wholedisk-bios-ipmi-direct-coreos-src http://logs.openstack.org/75/656275/3/check/ipa-tempest-dsvm-wholedisk-bios-ipmi-direct-coreos-src/31a661a/ : SUCCESS in 1h 07m 01s (non-voting)\n- openstack-tox-functional http://logs.openstack.org/75/656275/3/check/openstack-tox-functional/1b6ec30/ : SUCCESS in 2m 55s (non-voting)\n- openstack-tox-lower-constraints http://logs.openstack.org/75/656275/3/check/openstack-tox-lower-constraints/e205a00/ : SUCCESS in 3m 32s","accounts_in_message":[],"_revision_number":3},{"id":"f1ec13d64036bd3af5a76e768a59bd145e8403b2","author":{"_account_id":23851,"name":"Riccardo Pittau","email":"elfosardo@gmail.com","username":"elfosardo"},"date":"2019-04-30 09:20:25.000000000","message":"Patch Set 3:\n\nThanks, that looks like a big mistake in the description!\nCan you please add the link to the doc to the commit message?","accounts_in_message":[],"_revision_number":3},{"id":"f4527441f2ba69ff76e00a5cc601b185cf1eab6f","author":{"_account_id":12860,"name":"Dongcan Ye","email":"yedongcan@yeah.net","username":"yedongcan"},"date":"2019-04-30 09:38:12.000000000","message":"Patch Set 3:\n\n\u003e Thanks, that looks like a big mistake in the description!\n \u003e Can you please add the link to the doc to the commit message?\n\nThanks, I will update.","accounts_in_message":[],"_revision_number":3},{"id":"c78506bdf77ab412ae27634438e0f96034c89bf6","author":{"_account_id":12860,"name":"Dongcan Ye","email":"yedongcan@yeah.net","username":"yedongcan"},"date":"2019-04-30 09:45:42.000000000","message":"Patch Set 4: Commit message was updated.","accounts_in_message":[],"_revision_number":4},{"id":"20e97b2c2f998e15a5179d373d0b82233b9d33bc","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-04-30 11:21:53.000000000","message":"Patch Set 4: Verified+1\n\nBuild succeeded (check pipeline).\n\n- openstack-tox-pep8 http://logs.openstack.org/75/656275/4/check/openstack-tox-pep8/a8b1d93/ : SUCCESS in 3m 44s\n- openstack-tox-py27 http://logs.openstack.org/75/656275/4/check/openstack-tox-py27/17722db/ : SUCCESS in 3m 44s\n- openstack-tox-py36 http://logs.openstack.org/75/656275/4/check/openstack-tox-py36/78326f1/ : SUCCESS in 4m 04s\n- openstack-tox-cover http://logs.openstack.org/75/656275/4/check/openstack-tox-cover/5cd2f87/cover/ : SUCCESS in 2m 49s\n- openstack-tox-docs http://logs.openstack.org/75/656275/4/check/openstack-tox-docs/fa0e8b1/html/ : SUCCESS in 3m 29s\n- ipa-tempest-partition-bios-ipmi-direct-tinyipa-src http://logs.openstack.org/75/656275/4/check/ipa-tempest-partition-bios-ipmi-direct-tinyipa-src/db9e038/ : SUCCESS in 57m 09s\n- ipa-tempest-partition-bios-ipmi-iscsi-tinyipa-src http://logs.openstack.org/75/656275/4/check/ipa-tempest-partition-bios-ipmi-iscsi-tinyipa-src/1ff3e33/ : SUCCESS in 59m 31s\n- ipa-tempest-wholedisk-bios-ipmi-direct-tinyipa-src http://logs.openstack.org/75/656275/4/check/ipa-tempest-wholedisk-bios-ipmi-direct-tinyipa-src/56a412c/ : SUCCESS in 57m 14s\n- ipa-tempest-wholedisk-bios-ipmi-iscsi-tinyipa-src http://logs.openstack.org/75/656275/4/check/ipa-tempest-wholedisk-bios-ipmi-iscsi-tinyipa-src/a369673/ : SUCCESS in 50m 52s\n- ipa-tempest-ironic-inspector-src http://logs.openstack.org/75/656275/4/check/ipa-tempest-ironic-inspector-src/279f3e9/ : SUCCESS in 1h 32m 00s (non-voting)\n- ipa-tempest-dsvm-wholedisk-bios-ipmi-direct-coreos-src http://logs.openstack.org/75/656275/4/check/ipa-tempest-dsvm-wholedisk-bios-ipmi-direct-coreos-src/67b2283/ : FAILURE in 56m 23s (non-voting)\n- openstack-tox-functional http://logs.openstack.org/75/656275/4/check/openstack-tox-functional/9a7e98a/ : SUCCESS in 2m 41s (non-voting)\n- openstack-tox-lower-constraints http://logs.openstack.org/75/656275/4/check/openstack-tox-lower-constraints/c65ac7d/ : SUCCESS in 3m 29s","accounts_in_message":[],"_revision_number":4},{"id":"dd09451677b7492e5901a88a28ba83ee1ab86319","author":{"_account_id":24828,"name":"Kaifeng Wang","email":"kaifeng.w@gmail.com","username":"wangkf"},"date":"2019-05-01 06:08:38.000000000","message":"Patch Set 4: Code-Review-1\n\nAlthough 1-Bh is defined as implementation specific, I think according to description only channel 1-7 can be assigned to different types of communication media and protocols, in our case, lan channels, excerpted description:\n\n  Channels 1-7 can be assigned to different types types of communication media and protocols for IPMI messages (e.g. IPMB, LAN, ICMB, etc.), based on the system implementation.\n\nSo the question is whether we want to support a non-standard BMC?\nI am open to have the support, but we need to have a configuration option with default value of 7 to keep the standard behavior.","accounts_in_message":[],"_revision_number":4},{"id":"5ab2acfd70abdc34fa52722c980f6c663d1d1737","author":{"_account_id":23851,"name":"Riccardo Pittau","email":"elfosardo@gmail.com","username":"elfosardo"},"date":"2019-05-01 09:43:00.000000000","message":"Patch Set 4:\n\n@Kaifeng what if it\u0027s a huge overlook in the description? It\u0027s clear from the channel numbers that there are 11 channels and not just 7.\nThe latest revision is also quite old, 4 years I think, It might just have been forgotten.","accounts_in_message":[],"_revision_number":4},{"id":"4c7ddcf70807ea1d9846918b79ae09a162945bb9","author":{"_account_id":12860,"name":"Dongcan Ye","email":"yedongcan@yeah.net","username":"yedongcan"},"date":"2019-05-02 01:04:20.000000000","message":"Patch Set 4:\n\n@Kaifeng, thanks for review. As Riccardo Pittau said, the channel numbers that there are 11 channels and not just 7. It depends on baremetal product if they follows the IPMI standard.","accounts_in_message":[],"_revision_number":4},{"id":"6a47c37200a17a8d05018e5f16e2f0206305a89c","author":{"_account_id":24828,"name":"Kaifeng Wang","email":"kaifeng.w@gmail.com","username":"wangkf"},"date":"2019-05-05 06:53:49.000000000","message":"Patch Set 4:\n\nHi Riccardo, it\u0027s not how old it is, v2 r1.1 is still the latest spec we can see. I am not specialized on the spec, but I am cautious on the standard implementation.\n\nExcerpted from IPMI spec 1.7.26:\n\n A channel number identifies a particular connection. For example, 0 is the channel\n number for the primary IPMB. Up to nine total channels can be supported (the System Interface and primary\n IPMB, plus seven additional channels with a media type assigned by the implementer.)\n\nThis explains why there is a \"1-7\" in 6.3\n\nWhat bothers me is the comment in original code, it has an explicit statement on the channel numbers, I\u0027ll be keep concerning if there is no proof to support raising the default range.\n\nHi Dongcan, my concern is whether simply raising the range is following the spec, maybe we need a way around to address specific vendor implementation as mentioned in the last comment. Either way, this change will need a release note, could you append one?","accounts_in_message":[],"_revision_number":4},{"id":"e22a2129d3d4b35e1bf41290d6442e5eb40ac084","author":{"_account_id":12860,"name":"Dongcan Ye","email":"yedongcan@yeah.net","username":"yedongcan"},"date":"2019-05-05 10:25:05.000000000","message":"Patch Set 4:\n\n\u003e Hi Riccardo, it\u0027s not how old it is, v2 r1.1 is still the latest\n \u003e spec we can see. I am not specialized on the spec, but I am\n \u003e cautious on the standard implementation.\n \u003e \n \u003e Excerpted from IPMI spec 1.7.26:\n \u003e \n \u003e A channel number identifies a particular connection. For example, 0\n \u003e is the channel\n \u003e number for the primary IPMB. Up to nine total channels can be\n \u003e supported (the System Interface and primary\n \u003e IPMB, plus seven additional channels with a media type assigned by\n \u003e the implementer.)\n \u003e \n \u003e This explains why there is a \"1-7\" in 6.3\n \u003e \n \u003e What bothers me is the comment in original code, it has an explicit\n \u003e statement on the channel numbers, I\u0027ll be keep concerning if there\n \u003e is no proof to support raising the default range.\n \u003e \n \u003e Hi Dongcan, my concern is whether simply raising the range is\n \u003e following the spec, maybe we need a way around to address specific\n \u003e vendor implementation as mentioned in the last comment. Either way,\n \u003e this change will need a release note, could you append one?\n\nThanks Kaifeng. IMO, it just a comment, and the number can use 1-11, it depends on the specific IPMI implementation for a specific server system. Isn\u0027t it?\n\nI\u0027ll add a reno here, and wait for others\u0027 opinions.","accounts_in_message":[],"_revision_number":4},{"id":"51a47c2332f3a7a57ce113ab6352b5f287f5434e","author":{"_account_id":12860,"name":"Dongcan Ye","email":"yedongcan@yeah.net","username":"yedongcan"},"date":"2019-05-05 10:27:50.000000000","message":"Uploaded patch set 5.","accounts_in_message":[],"_revision_number":5},{"id":"057325490f0ea4a589f4d648cd393b28770dc8db","author":{"_account_id":12860,"name":"Dongcan Ye","email":"yedongcan@yeah.net","username":"yedongcan"},"date":"2019-05-05 10:30:19.000000000","message":"Uploaded patch set 6.","accounts_in_message":[],"_revision_number":6},{"id":"0623e9e46934d26e10193be4a84a5556fb88f0c6","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-05-05 12:28:16.000000000","message":"Patch Set 6: Verified+1\n\nBuild succeeded (check pipeline).\n\n- openstack-tox-pep8 http://logs.openstack.org/75/656275/6/check/openstack-tox-pep8/1b028ec/ : SUCCESS in 3m 58s\n- openstack-tox-py27 http://logs.openstack.org/75/656275/6/check/openstack-tox-py27/e581f79/ : SUCCESS in 3m 07s\n- openstack-tox-py36 http://logs.openstack.org/75/656275/6/check/openstack-tox-py36/7c939da/ : SUCCESS in 3m 23s\n- openstack-tox-cover http://logs.openstack.org/75/656275/6/check/openstack-tox-cover/90e4407/cover/ : SUCCESS in 3m 29s\n- openstack-tox-docs http://logs.openstack.org/75/656275/6/check/openstack-tox-docs/7695fa3/html/ : SUCCESS in 2m 37s\n- build-openstack-releasenotes http://logs.openstack.org/75/656275/6/check/build-openstack-releasenotes/d13d3cb/html/ : SUCCESS in 2m 24s\n- ipa-tempest-partition-bios-ipmi-direct-tinyipa-src http://logs.openstack.org/75/656275/6/check/ipa-tempest-partition-bios-ipmi-direct-tinyipa-src/1db1ce5/ : SUCCESS in 53m 12s\n- ipa-tempest-partition-bios-ipmi-iscsi-tinyipa-src http://logs.openstack.org/75/656275/6/check/ipa-tempest-partition-bios-ipmi-iscsi-tinyipa-src/5b7188b/ : SUCCESS in 1h 05m 15s\n- ipa-tempest-wholedisk-bios-ipmi-direct-tinyipa-src http://logs.openstack.org/75/656275/6/check/ipa-tempest-wholedisk-bios-ipmi-direct-tinyipa-src/654a058/ : SUCCESS in 1h 17m 23s\n- ipa-tempest-wholedisk-bios-ipmi-iscsi-tinyipa-src http://logs.openstack.org/75/656275/6/check/ipa-tempest-wholedisk-bios-ipmi-iscsi-tinyipa-src/e22ad6c/ : SUCCESS in 53m 32s\n- ipa-tempest-ironic-inspector-src http://logs.openstack.org/75/656275/6/check/ipa-tempest-ironic-inspector-src/ca28d04/ : SUCCESS in 1h 56m 56s (non-voting)\n- ipa-tempest-dsvm-wholedisk-bios-ipmi-direct-coreos-src http://logs.openstack.org/75/656275/6/check/ipa-tempest-dsvm-wholedisk-bios-ipmi-direct-coreos-src/3aad444/ : SUCCESS in 46m 00s (non-voting)\n- openstack-tox-functional http://logs.openstack.org/75/656275/6/check/openstack-tox-functional/7fd0f1c/ : SUCCESS in 2m 59s (non-voting)\n- openstack-tox-lower-constraints http://logs.openstack.org/75/656275/6/check/openstack-tox-lower-constraints/ca54acf/ : SUCCESS in 4m 14s","accounts_in_message":[],"_revision_number":6},{"id":"48b516ad77b9b7f30f43e9b63f0f2942ffd0b61f","author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"date":"2019-05-07 15:43:02.000000000","message":"Patch Set 6: Code-Review+2","accounts_in_message":[],"_revision_number":6},{"id":"23381126aeff787cc8489c96cad50b2b7ae45358","author":{"_account_id":11292,"name":"Arne Wiebalck","email":"Arne.Wiebalck@cern.ch","username":"wiebalck"},"date":"2019-05-07 16:07:26.000000000","message":"Patch Set 6:\n\nKaifeng, doesn\u0027t \"IPMI Specification, V2.0/1.5, Rev. 4: Addendum\" clarify this? It corrects \u00271-7\u0027 to \u00271-Bh\u0027 which seems like there was either a mistake initially or the range has been extended. Either way, I think we can assume that if the \"added\" channels were meant for a different purpose, there would be an additional entry in that table, no?","accounts_in_message":[],"_revision_number":6},{"id":"5164f8ef1ad037af8d3be395f94d5c9d96e93928","author":{"_account_id":24828,"name":"Kaifeng Wang","email":"kaifeng.w@gmail.com","username":"wangkf"},"date":"2019-05-08 00:59:25.000000000","message":"Patch Set 6:\n\n\u003e Kaifeng, doesn\u0027t \"IPMI Specification, V2.0/1.5, Rev. 4: Addendum\"\n \u003e clarify this? It corrects \u00271-7\u0027 to \u00271-Bh\u0027 which seems like there\n \u003e was either a mistake initially or the range has been extended.\n \u003e Either way, I think we can assume that if the \"added\" channels were\n \u003e meant for a different purpose, there would be an additional entry\n \u003e in that table, no?\n\nThanks Arne, I am convinced :)\nI wonder why the spec didn\u0027t update the description as well..\n\nFor reference:\nhttps://www.intel.com/content/www/us/en/servers/ipmi/second-gen-interface-spec-v4.html\nP.87","accounts_in_message":[],"_revision_number":6},{"id":"09fc8b6caad94a9c680f364f0e1d99d408b5ef25","author":{"_account_id":24828,"name":"Kaifeng Wang","email":"kaifeng.w@gmail.com","username":"wangkf"},"date":"2019-05-08 01:01:01.000000000","message":"Patch Set 6: Code-Review+2 Workflow+1\n\n(1 comment)","accounts_in_message":[],"_revision_number":6},{"id":"55a2a9f45aa48ecb98fa1464a21d6b31480f4d8f","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-05-08 01:01:13.000000000","message":"Patch Set 6: -Verified\n\nStarting gate jobs.","accounts_in_message":[],"_revision_number":6},{"id":"1a75a4b3bcee7897fb92436a8b929c20a842460c","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-05-08 02:03:44.000000000","message":"Change has been successfully merged by Zuul","accounts_in_message":[],"_revision_number":6},{"id":"b424162625d57de86e9d63dfdea0ba44c2cb537b","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-05-08 02:03:44.000000000","message":"Patch Set 6: Verified+2\n\nBuild succeeded (gate pipeline).\n\n- openstack-tox-pep8 http://logs.openstack.org/75/656275/6/gate/openstack-tox-pep8/186469c/ : SUCCESS in 3m 18s\n- openstack-tox-py27 http://logs.openstack.org/75/656275/6/gate/openstack-tox-py27/ad54acc/ : SUCCESS in 3m 25s\n- openstack-tox-py36 http://logs.openstack.org/75/656275/6/gate/openstack-tox-py36/703769a/ : SUCCESS in 3m 17s\n- openstack-tox-docs http://logs.openstack.org/75/656275/6/gate/openstack-tox-docs/b0cdb6f/html/ : SUCCESS in 3m 20s\n- build-openstack-releasenotes http://logs.openstack.org/75/656275/6/gate/build-openstack-releasenotes/dcc48a0/html/ : SUCCESS in 2m 56s\n- ipa-tempest-partition-bios-ipmi-direct-tinyipa-src http://logs.openstack.org/75/656275/6/gate/ipa-tempest-partition-bios-ipmi-direct-tinyipa-src/1709925/ : SUCCESS in 55m 07s\n- ipa-tempest-partition-bios-ipmi-iscsi-tinyipa-src http://logs.openstack.org/75/656275/6/gate/ipa-tempest-partition-bios-ipmi-iscsi-tinyipa-src/58f0049/ : SUCCESS in 1h 01m 09s\n- ipa-tempest-wholedisk-bios-ipmi-direct-tinyipa-src http://logs.openstack.org/75/656275/6/gate/ipa-tempest-wholedisk-bios-ipmi-direct-tinyipa-src/a0d989e/ : SUCCESS in 59m 11s\n- ipa-tempest-wholedisk-bios-ipmi-iscsi-tinyipa-src http://logs.openstack.org/75/656275/6/gate/ipa-tempest-wholedisk-bios-ipmi-iscsi-tinyipa-src/1f9ff47/ : SUCCESS in 1h 00m 14s\n- openstack-tox-lower-constraints http://logs.openstack.org/75/656275/6/gate/openstack-tox-lower-constraints/3227fd4/ : SUCCESS in 3m 26s","accounts_in_message":[],"_revision_number":6},{"id":"db36b27321ce0e2b7639101aa7d0ef97a3ac40d9","author":{"_account_id":12860,"name":"Dongcan Ye","email":"yedongcan@yeah.net","username":"yedongcan"},"date":"2019-05-08 02:06:12.000000000","message":"Patch Set 6:\n\n@Arne, Kaifeng, thanks for clarify.","accounts_in_message":[],"_revision_number":6},{"id":"56345e388e9547c9d5e93a153ef8bc187d70d539","author":{"_account_id":12860,"name":"Dongcan Ye","email":"yedongcan@yeah.net","username":"yedongcan"},"date":"2019-05-08 02:06:29.000000000","message":"Patch Set 6: Cherry Picked\n\nThis patchset was cherry picked to branch stable/stein as commit b2f0539d9ca223e5186d05bd7f4a86a162bc377d","accounts_in_message":[],"_revision_number":6},{"id":"290e7b3494f6f37acff0f221766fc54fc96406e5","author":{"_account_id":12860,"name":"Dongcan Ye","email":"yedongcan@yeah.net","username":"yedongcan"},"date":"2019-05-08 02:06:49.000000000","message":"Patch Set 6: Cherry Picked\n\nThis patchset was cherry picked to branch stable/rocky as commit b8f602635fb2caba70afc6a29f246d92a81fc263","accounts_in_message":[],"_revision_number":6}],"current_revision_number":6,"current_revision":"66d3c8e453a58c01740581079045e9b4368c4bde","revisions":{"6455143888072fda0acb65180011964945ad5fb1":{"kind":"REWORK","_number":1,"created":"2019-04-29 09:33:16.000000000","uploader":{"_account_id":12860,"name":"Dongcan Ye","email":"yedongcan@yeah.net","username":"yedongcan"},"ref":"refs/changes/75/656275/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/ironic-python-agent","ref":"refs/changes/75/656275/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/ironic-python-agent refs/changes/75/656275/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/ironic-python-agent refs/changes/75/656275/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/ironic-python-agent refs/changes/75/656275/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/ironic-python-agent refs/changes/75/656275/1"}}},"commit":{"parents":[{"commit":"73208b48b73f3f4b8f0b77a65699a37cd21bff0e","subject":"Merge \"Bind mount /run into chroot when installing grub\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ironic-python-agent/commit/73208b48b73f3f4b8f0b77a65699a37cd21bff0e"}]}],"author":{"name":"Dongcan Ye","email":"hellochosen@gmail.com","date":"2019-04-29 09:29:18.000000000","tz":0},"committer":{"name":"Dongcan Ye","email":"hellochosen@gmail.com","date":"2019-04-29 09:29:18.000000000","tz":0},"subject":"Add more channel model for detecting BMC IP address","message":"Add more channel model for detecting BMC IP address\n\nIPMI SPEC assignment channel number 1-14 for vendors\nimplementation, we met an baremetal product,\nlike H3C, use number 8 as LAN channel.\n\nCurrent code limit channel number to 1~7,\nI think this can increase to 1~14.\n\nChange-Id: I63a6ff91d702b990ef09da13e6e2e269e7274cce\nTask: 30654\nStory: 2005528\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ironic-python-agent/commit/6455143888072fda0acb65180011964945ad5fb1"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ironic-python-agent/commit/6455143888072fda0acb65180011964945ad5fb1"}]},"branch":"refs/heads/master"},"524a3d245600fe83c57c98926aa3fb51306e911d":{"kind":"NO_CODE_CHANGE","_number":2,"created":"2019-04-29 09:35:41.000000000","uploader":{"_account_id":12860,"name":"Dongcan Ye","email":"yedongcan@yeah.net","username":"yedongcan"},"ref":"refs/changes/75/656275/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/ironic-python-agent","ref":"refs/changes/75/656275/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/ironic-python-agent refs/changes/75/656275/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/ironic-python-agent refs/changes/75/656275/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/ironic-python-agent refs/changes/75/656275/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/ironic-python-agent refs/changes/75/656275/2"}}},"commit":{"parents":[{"commit":"73208b48b73f3f4b8f0b77a65699a37cd21bff0e","subject":"Merge \"Bind mount /run into chroot when installing grub\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ironic-python-agent/commit/73208b48b73f3f4b8f0b77a65699a37cd21bff0e"}]}],"author":{"name":"Dongcan Ye","email":"hellochosen@gmail.com","date":"2019-04-29 09:29:18.000000000","tz":0},"committer":{"name":"Dongcan Ye","email":"hellochosen@gmail.com","date":"2019-04-29 09:34:30.000000000","tz":0},"subject":"Add more channel number for detecting BMC IP address","message":"Add more channel number for detecting BMC IP address\n\nIPMI SPEC assignment channel number 1-14 for vendors\nimplementation, we met an baremetal product,\nlike H3C, use number 8 as LAN channel.\n\nCurrent code limit channel number to 1~7,\nI think this can increase to 1~14.\n\nChange-Id: I63a6ff91d702b990ef09da13e6e2e269e7274cce\nTask: 30654\nStory: 2005528\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ironic-python-agent/commit/524a3d245600fe83c57c98926aa3fb51306e911d"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ironic-python-agent/commit/524a3d245600fe83c57c98926aa3fb51306e911d"}]},"branch":"refs/heads/master"},"1153c98ec8f28464c5257d4ba4b8b1a232e3fbdc":{"kind":"NO_CODE_CHANGE","_number":3,"created":"2019-04-29 10:30:00.000000000","uploader":{"_account_id":12860,"name":"Dongcan Ye","email":"yedongcan@yeah.net","username":"yedongcan"},"ref":"refs/changes/75/656275/3","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/ironic-python-agent","ref":"refs/changes/75/656275/3","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/ironic-python-agent refs/changes/75/656275/3 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/ironic-python-agent refs/changes/75/656275/3 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/ironic-python-agent refs/changes/75/656275/3 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/ironic-python-agent refs/changes/75/656275/3"}}},"commit":{"parents":[{"commit":"73208b48b73f3f4b8f0b77a65699a37cd21bff0e","subject":"Merge \"Bind mount /run into chroot when installing grub\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ironic-python-agent/commit/73208b48b73f3f4b8f0b77a65699a37cd21bff0e"}]}],"author":{"name":"Dongcan Ye","email":"hellochosen@gmail.com","date":"2019-04-29 09:29:18.000000000","tz":0},"committer":{"name":"Dongcan Ye","email":"yedongcan@yeah.net","date":"2019-04-29 10:29:56.000000000","tz":0},"subject":"Add more channel number for detecting BMC IP address","message":"Add more channel number for detecting BMC IP address\n\nIPMI SPEC assignment channel number 1-11 for vendors\nimplementation, we met an baremetal product,\nlike H3C, use number 8 as LAN channel.\n\nCurrent code limit channel number to 1~7,\nI think this can increase to 1~11.\n\nChange-Id: I63a6ff91d702b990ef09da13e6e2e269e7274cce\nTask: 30654\nStory: 2005528\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ironic-python-agent/commit/1153c98ec8f28464c5257d4ba4b8b1a232e3fbdc"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ironic-python-agent/commit/1153c98ec8f28464c5257d4ba4b8b1a232e3fbdc"}]},"branch":"refs/heads/master"},"088c03b05d2de9ee2ffd9b830568a970f28a676e":{"kind":"NO_CODE_CHANGE","_number":4,"created":"2019-04-30 09:45:42.000000000","uploader":{"_account_id":12860,"name":"Dongcan Ye","email":"yedongcan@yeah.net","username":"yedongcan"},"ref":"refs/changes/75/656275/4","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/ironic-python-agent","ref":"refs/changes/75/656275/4","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/ironic-python-agent refs/changes/75/656275/4 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/ironic-python-agent refs/changes/75/656275/4 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/ironic-python-agent refs/changes/75/656275/4 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/ironic-python-agent refs/changes/75/656275/4"}}},"commit":{"parents":[{"commit":"73208b48b73f3f4b8f0b77a65699a37cd21bff0e","subject":"Merge \"Bind mount /run into chroot when installing grub\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ironic-python-agent/commit/73208b48b73f3f4b8f0b77a65699a37cd21bff0e"}]}],"author":{"name":"Dongcan Ye","email":"hellochosen@gmail.com","date":"2019-04-29 09:29:18.000000000","tz":0},"committer":{"name":"Dongcan Ye","email":"yedongcan@yeah.net","date":"2019-04-30 09:45:31.000000000","tz":0},"subject":"Add more channel number for detecting BMC IP address","message":"Add more channel number for detecting BMC IP address\n\nIPMI SPEC[1] assignment channel number 1-11 for vendors\nimplementation(We an search \"Channel Numbers\" in section 6.3).\nWe met an baremetal product, like H3C, use number 8 as LAN channel.\n\nCurrent code limit channel number to 1~7,\nI think this can increase to 1~11.\n\n[1] https://www.intel.com/content/dam/www/public/us/en/documents/product-briefs/ipmi-second-gen-interface-spec-v2-rev1-1.pdf\n\nChange-Id: I63a6ff91d702b990ef09da13e6e2e269e7274cce\nTask: 30654\nStory: 2005528\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ironic-python-agent/commit/088c03b05d2de9ee2ffd9b830568a970f28a676e"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ironic-python-agent/commit/088c03b05d2de9ee2ffd9b830568a970f28a676e"}]},"branch":"refs/heads/master"},"c576655a60e7e133288611123f097615e8c3286c":{"kind":"REWORK","_number":5,"created":"2019-05-05 10:27:50.000000000","uploader":{"_account_id":12860,"name":"Dongcan Ye","email":"yedongcan@yeah.net","username":"yedongcan"},"ref":"refs/changes/75/656275/5","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/ironic-python-agent","ref":"refs/changes/75/656275/5","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/ironic-python-agent refs/changes/75/656275/5 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/ironic-python-agent refs/changes/75/656275/5 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/ironic-python-agent refs/changes/75/656275/5 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/ironic-python-agent refs/changes/75/656275/5"}}},"commit":{"parents":[{"commit":"73208b48b73f3f4b8f0b77a65699a37cd21bff0e","subject":"Merge \"Bind mount /run into chroot when installing grub\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ironic-python-agent/commit/73208b48b73f3f4b8f0b77a65699a37cd21bff0e"}]}],"author":{"name":"Dongcan Ye","email":"hellochosen@gmail.com","date":"2019-04-29 09:29:18.000000000","tz":0},"committer":{"name":"Dongcan Ye","email":"hellochosen@gmail.com","date":"2019-05-05 10:26:04.000000000","tz":0},"subject":"Add more channel number for detecting BMC IP address","message":"Add more channel number for detecting BMC IP address\n\nIPMI SPEC[1] assignment channel number 1-11 for vendors\nimplementation(We an search \"Channel Numbers\" in section 6.3).\nWe met an baremetal product, like H3C, use number 8 as LAN channel.\n\nCurrent code limit channel number to 1~7,\nI think this can increase to 1~11.\n\n[1] https://www.intel.com/content/dam/www/public/us/en/documents/product-briefs/ipmi-second-gen-interface-spec-v2-rev1-1.pdf\n\nChange-Id: I63a6ff91d702b990ef09da13e6e2e269e7274cce\nTask: 30654\nStory: 2005528\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ironic-python-agent/commit/c576655a60e7e133288611123f097615e8c3286c"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ironic-python-agent/commit/c576655a60e7e133288611123f097615e8c3286c"}]},"branch":"refs/heads/master"},"66d3c8e453a58c01740581079045e9b4368c4bde":{"kind":"REWORK","_number":6,"created":"2019-05-05 10:30:19.000000000","uploader":{"_account_id":12860,"name":"Dongcan Ye","email":"yedongcan@yeah.net","username":"yedongcan"},"ref":"refs/changes/75/656275/6","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/ironic-python-agent","ref":"refs/changes/75/656275/6","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/ironic-python-agent refs/changes/75/656275/6 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/ironic-python-agent refs/changes/75/656275/6 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/ironic-python-agent refs/changes/75/656275/6 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/ironic-python-agent refs/changes/75/656275/6"}}},"commit":{"parents":[{"commit":"73208b48b73f3f4b8f0b77a65699a37cd21bff0e","subject":"Merge \"Bind mount /run into chroot when installing grub\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ironic-python-agent/commit/73208b48b73f3f4b8f0b77a65699a37cd21bff0e"}]}],"author":{"name":"Dongcan Ye","email":"hellochosen@gmail.com","date":"2019-04-29 09:29:18.000000000","tz":0},"committer":{"name":"Dongcan Ye","email":"hellochosen@gmail.com","date":"2019-05-05 10:29:58.000000000","tz":0},"subject":"Add more channel number for detecting BMC IP address","message":"Add more channel number for detecting BMC IP address\n\nIPMI SPEC[1] assignment channel number 1-11 for vendors\nimplementation(We an search \"Channel Numbers\" in section 6.3).\nWe met an baremetal product, like H3C, use number 8 as LAN channel.\n\nCurrent code limit channel number to 1~7,\nI think this can increase to 1~11.\n\n[1] https://www.intel.com/content/dam/www/public/us/en/documents/product-briefs/ipmi-second-gen-interface-spec-v2-rev1-1.pdf\n\nChange-Id: I63a6ff91d702b990ef09da13e6e2e269e7274cce\nTask: 30654\nStory: 2005528\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ironic-python-agent/commit/66d3c8e453a58c01740581079045e9b4368c4bde"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ironic-python-agent/commit/66d3c8e453a58c01740581079045e9b4368c4bde"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}