[03:26:38] Project autotest-nightly build #450: STILL FAILING in 22 min. See https://ci.spdk.io/spdk-jenkins for results. [07:21:02] *** Joins: felipef (~felipef@62.254.189.133) [07:41:26] hi jrlusby - yes, our next bug scrub is this evening - 9pm PDT [07:42:15] fyi - you can always see dates/times for the next bug scrub and community meetings at https://spdk.io/community/ [07:44:37] if an spdk subsystem fails to initialize, do we want that subsystem to be de-initialized with its fini callback? [07:44:43] from a high level perspective [07:45:47] we seem to have issues with that atm. Failing to init the nvmf subsystem is totally broken [07:46:26] oh - you mean if subsystem A fails to init(), do we want to call subsystem A's fini()? [07:46:44] yeah [07:47:17] i guess my initial thinking is that a subsystem is responsible for cleaning itself up if it fails init() [07:47:29] i.e. we only call fini() on successfully initialized subsystems [07:47:54] yeah that's what i was thinking too [07:48:01] but we don't do that in the bdev subsystem [07:48:20] if any bdev module fails to initialize, we just bail from init() immediately [07:49:08] yeah - there's what I think it should do, and then what it actually does :-) [07:49:51] NVMf is one, but I can guess there are more subsystems which don't handle their initialization failures properly [07:49:59] as we don't test that on CI [07:54:18] sethhowe: some of the links don't seem to be working when reviewing patches to spdk.github.io [07:54:20] https://dqtibwqq6s6ux.cloudfront.net/results/spdk-github-io/builds/17/archive/spdk-github-io-test/_site/index.html [07:54:31] this works, but when I click on Community I get some access denied message [08:02:46] I created a github issue for that error handling in spdk subsystems - https://github.com/spdk/spdk/issues/739 [09:00:12] jimharris: oh shoot. I didn't think about that. https://dqtibwqq6s6ux.cloudfront.net/results/spdk-github-io/builds/17/archive/spdk-github-io-test/_site/community/index.html works. [09:02:47] In a webserver, if you specify a directory in a url, it will automatically look at the link and give you the index page in that directory. S3 doesn't support this. However, there is a way around it using AWS. [09:04:48] I can fix something up to make that traversal seamless, but in the meantime, when reviewing the status page links, just add index.html onto the end of any directory paths and you will get the proper page. [09:05:40] works for me [09:05:44] thanks! [10:31:42] *** Quits: gila (~gila@94-212-217-121.cable.dynamic.v4.ziggo.nl) (Ping timeout: 246 seconds) [10:32:13] *** Joins: gila (~gila@94-212-217-121.cable.dynamic.v4.ziggo.nl) [10:35:57] jimharris: The AWS fix is up now. You should be able to navigate through the links without having to append any URLs. Thanks for pointing that out. [11:11:16] *** Quits: felipef (~felipef@62.254.189.133) (Remote host closed the connection) [11:11:50] *** Joins: felipef (~felipef@62.254.189.133) [11:15:43] *** Joins: MaZ- (~maz@dont.feed.the.lawlr.us) [11:16:18] *** Quits: felipef (~felipef@62.254.189.133) (Ping timeout: 250 seconds) [14:13:03] *** Joins: felipef (~felipef@cpc92310-cmbg19-2-0-cust421.5-4.cable.virginm.net) [14:17:27] *** Quits: felipef (~felipef@cpc92310-cmbg19-2-0-cust421.5-4.cable.virginm.net) (Ping timeout: 246 seconds) [18:15:05] *** Joins: travis-ci (~travis-ci@ec2-54-224-18-35.compute-1.amazonaws.com) [18:15:06] (spdk/master) examples/hello_blob: use io_unit_size instead of page_size (Jim Harris) [18:15:06] Diff URL: https://github.com/spdk/spdk/compare/4fad4b86dd6e...1a2de8245655 [18:15:06] *** Parts: travis-ci (~travis-ci@ec2-54-224-18-35.compute-1.amazonaws.com) () [18:37:05] *** Joins: ziyeyang_ (ziyeyang@nat/intel/x-ejfuqxqsyrglhfns) [18:46:45] *** Joins: travis-ci (~travis-ci@ec2-3-91-144-243.compute-1.amazonaws.com) [18:46:46] (spdk/master) nvmf: fix reservation acquire typo (Liang Yan) [18:46:46] Diff URL: https://github.com/spdk/spdk/compare/0acb123aa491...ad08de311ef3 [18:46:46] *** Parts: travis-ci (~travis-ci@ec2-3-91-144-243.compute-1.amazonaws.com) () [18:48:35] *** Joins: travis-ci (~travis-ci@ec2-3-85-230-237.compute-1.amazonaws.com) [18:48:36] (spdk/master) app/trace: check num_entries before referring (Xiaodong Liu) [18:48:36] Diff URL: https://github.com/spdk/spdk/compare/ad08de311ef3...1468f8dd3ba9 [18:48:36] *** Parts: travis-ci (~travis-ci@ec2-3-85-230-237.compute-1.amazonaws.com) () [21:03:31] *** Joins: travis-ci (~travis-ci@54.234.208.179) [21:03:32] (spdk/master) test/iscsi: remove leftover code from fio.sh (Karol Latecki) [21:03:32] Diff URL: https://github.com/spdk/spdk/compare/1468f8dd3ba9...c1dd96ae6329 [21:03:32] *** Parts: travis-ci (~travis-ci@54.234.208.179) () [21:05:09] *** Joins: travis-ci (~travis-ci@ec2-54-234-208-179.compute-1.amazonaws.com) [21:05:09] (spdk/master) scripts, test: fix python overindent issues (shahar salzman) [21:05:09] Diff URL: https://github.com/spdk/spdk/compare/c1dd96ae6329...21cfc857ff10 [21:05:09] *** Parts: travis-ci (~travis-ci@ec2-54-234-208-179.compute-1.amazonaws.com) () [21:39:36] *** Joins: travis-ci (~travis-ci@ec2-3-85-230-237.compute-1.amazonaws.com) [21:39:37] (spdk/master) rdma: reorder qpair elements to plug hole (Seth Howell) [21:39:37] Diff URL: https://github.com/spdk/spdk/compare/21cfc857ff10...9f7582c3a520 [21:39:37] *** Parts: travis-ci (~travis-ci@ec2-3-85-230-237.compute-1.amazonaws.com) () [21:45:06] *** Joins: travis-ci (~travis-ci@ec2-3-91-144-243.compute-1.amazonaws.com) [21:45:07] (spdk/master) iscsi: Fix LUN ID in Ready to Transfer PDUs (Shuhei Matsumoto) [21:45:07] Diff URL: https://github.com/spdk/spdk/compare/9f7582c3a520...5a1dc44423e1 [21:45:07] *** Parts: travis-ci (~travis-ci@ec2-3-91-144-243.compute-1.amazonaws.com) () [21:54:38] *** Joins: travis-ci (~travis-ci@ec2-3-91-144-243.compute-1.amazonaws.com) [21:54:38] (spdk/master) nvme/tcp: Add a helper function nvme_tcp_pdu_set_data (Ziye Yang) [21:54:38] Diff URL: https://github.com/spdk/spdk/compare/5a1dc44423e1...8f3b4a3a6d41 [21:54:38] *** Parts: travis-ci (~travis-ci@ec2-3-91-144-243.compute-1.amazonaws.com) () [22:59:34] *** Joins: travis-ci (~travis-ci@ec2-3-83-244-158.compute-1.amazonaws.com) [22:59:35] (spdk/master) nvmf: Use threads instead of events for shutdown handling (Ben Walker) [22:59:35] Diff URL: https://github.com/spdk/spdk/compare/8f3b4a3a6d41...a0760b9554a2 [22:59:35] *** Parts: travis-ci (~travis-ci@ec2-3-83-244-158.compute-1.amazonaws.com) () [23:08:07] Project autotest-nightly build #451: STILL FAILING in 29 min. See https://ci.spdk.io/spdk-jenkins for results. [23:24:55] Project autotest-nightly-failing build #318: STILL FAILING in 40 min. See https://ci.spdk.io/spdk-jenkins for results. [23:37:28] *** Quits: gila (~gila@94-212-217-121.cable.dynamic.v4.ziggo.nl) () [23:37:45] *** Joins: gila (~gila@94-212-217-121.cable.dynamic.v4.ziggo.nl)