{"push_id":2369333918,"size":32,"distinct_size":32,"ref":"refs/heads/master-cache-dbus","head":"8278b49bf63c8e591f6718ac77dd90270587635c","before":"f61e522ac0046f5371f28209639498db979d21e0","commits":[{"sha":"efad1e030255e56748d8e7ae2aaabb04a1e39a87","author":{"name":"Todd Gill","email":"278dbe5a0cb4e26d55366afacf555c87825371ab@redhat.com"},"message":"SeekFrom::Start() takes bytes as a parameter (not Sectors)\n\nCurrently only called with an offset of 0.\n\nSigned-off-by: Todd Gill <tgill@redhat.com>","distinct":true,"url":"https://api.github.com/repos/mulkieran/stratisd/commits/efad1e030255e56748d8e7ae2aaabb04a1e39a87"},{"sha":"5f7bc5105574a17add2eac5569e12e21addd22be","author":{"name":"Todd Gill","email":"278dbe5a0cb4e26d55366afacf555c87825371ab@redhat.com"},"message":"Use available space if double isn't available\n\nIf there is insufficient space available to double the physical space\nallocated to a pool.  Request the remaining available space.\n\nSigned-off-by: Todd Gill <tgill@redhat.com>","distinct":true,"url":"https://api.github.com/repos/mulkieran/stratisd/commits/5f7bc5105574a17add2eac5569e12e21addd22be"},{"sha":"405d00743f86a36f0420cf4bc6c87c6ece016467","author":{"name":"Andy Grover","email":"3b57b84c25fafa432787cb19a4e06e8a6bd56948@groveronline.com"},"message":"Merge pull request #803 from trgill/partial_expand_data\n\nUse available space if double isn't available","distinct":true,"url":"https://api.github.com/repos/mulkieran/stratisd/commits/405d00743f86a36f0420cf4bc6c87c6ece016467"},{"sha":"1fd2bd9e5c2cc130e345f2843da7c1f1b7fcc0e3","author":{"name":"Andy Grover","email":"3b57b84c25fafa432787cb19a4e06e8a6bd56948@groveronline.com"},"message":"Merge pull request #802 from trgill/fix_write_sectors_offset\n\nSeekFrom::Start() takes bytes as a parameter (not Sectors)","distinct":true,"url":"https://api.github.com/repos/mulkieran/stratisd/commits/1fd2bd9e5c2cc130e345f2843da7c1f1b7fcc0e3"},{"sha":"197e0efcdf28aaec8b8092b374959991e8a19cd3","author":{"name":"mulhern","email":"87548b8fc7ce9ff25c34053404638f744f7df56b@redhat.com"},"message":"Introduce cachedevs into the D-Bus layer\n\n1. Add a new BlockDevTier property to dbus blockdev interface\n\nThis will allow to distinguish between cache and data devs.\nExtend Pool::get_blockdev and Pool::get_mut_blockdev so that they return\nthe tier of the located blockdev.\n\nThis approach keeps the D-Bus and engine layer separate.\nThe alternative approach, which extends the Pool interface with additional\nmethods, entangles the two layers more, as the D-Bus layer has to put more\nmeaning into which methods it calls.\n\n2. Add a BlockDevTier parameter to Pool::add_blockdevs\n\nReturn an error if BlockDevTier::Cache is passed, because no code yet\nexists to add the cache blockdevs.\n\n3. Add an AddCacheDevs method to the D-Bus API\n\nChange AddDevs to add AddDataDevs for easier distinguishing of the two.\n\nShare the implementation the methods add_cachedevs and add_blockdevs in the\nAPI. This may be a temporary thing.\n\nThe idea of having AddCacheDevs be a separate method in the D-Bus API is\nthat at some point the parameters of AddCacheDevs and AddBlockDevs might\ndiverge.  On the other hand, they haven't yet, so it is somewhat reasonable\nto share the internal implementation. It is also very cheap in programmer\ntime, so it is a reasonable way to start.\n\nSigned-off-by: mulhern <amulhern@redhat.com>","distinct":true,"url":"https://api.github.com/repos/mulkieran/stratisd/commits/197e0efcdf28aaec8b8092b374959991e8a19cd3"},{"sha":"6a6a79f29dd3759c8bad0ac41bbe15f64204b324","author":{"name":"mulhern","email":"87548b8fc7ce9ff25c34053404638f744f7df56b@redhat.com"},"message":"Specialize some Backstore methods to datadev only\n\nTo distinguish between methods that are for all blockdevs, just for data\ndevs, or just for cachedevs.\n\nSigned-off-by: mulhern <amulhern@redhat.com>","distinct":true,"url":"https://api.github.com/repos/mulkieran/stratisd/commits/6a6a79f29dd3759c8bad0ac41bbe15f64204b324"},{"sha":"9268366d27a9a937021aa6d4415af76689a83127","author":{"name":"mulhern","email":"87548b8fc7ce9ff25c34053404638f744f7df56b@redhat.com"},"message":"Add a clarifying comment for Backstore::blockdevs\n\nIt really means all blockdevs.\n\nSigned-off-by: mulhern <amulhern@redhat.com>","distinct":true,"url":"https://api.github.com/repos/mulkieran/stratisd/commits/9268366d27a9a937021aa6d4415af76689a83127"},{"sha":"0ca0a57648b5f0c1c535d5753f14c63b53eacd57","author":{"name":"mulhern","email":"87548b8fc7ce9ff25c34053404638f744f7df56b@redhat.com"},"message":"Move DM device out of data layer\n\nManage allocation from the DM device in the Backstore.\nWhen cache functionality is added, this will be necessary for a number of\nreasons.\n\n1. dm_device will have to change from Linear to Cache\n2. dm_device will transition to being owned by the CacheDev\n3. allocation will be from the DM not from the blockdevs, or which there\nwill be only allocated from.\n\nSigned-off-by: mulhern <amulhern@redhat.com>","distinct":true,"url":"https://api.github.com/repos/mulkieran/stratisd/commits/0ca0a57648b5f0c1c535d5753f14c63b53eacd57"},{"sha":"81c102dde93bedec649b49bc7b49c45994daf416","author":{"name":"mulhern","email":"87548b8fc7ce9ff25c34053404638f744f7df56b@redhat.com"},"message":"Move setting the tier of a looked-up blockdev into Backstore\n\nWe know that blockdevs retrieved from the data tier are always data\nblockdevs, so set the value there.\n\nSigned-off-by: mulhern <amulhern@redhat.com>","distinct":true,"url":"https://api.github.com/repos/mulkieran/stratisd/commits/81c102dde93bedec649b49bc7b49c45994daf416"},{"sha":"294e4d1a1e66464db0527c5b1d2421484f3ef7c6","author":{"name":"mulhern","email":"87548b8fc7ce9ff25c34053404638f744f7df56b@redhat.com"},"message":"Add an optional cache tier that does nothing\n\nThis is just a staging step, and it all compiles.\n\nSigned-off-by: mulhern <amulhern@redhat.com>","distinct":true,"url":"https://api.github.com/repos/mulkieran/stratisd/commits/294e4d1a1e66464db0527c5b1d2421484f3ef7c6"},{"sha":"2bea60864bd72db1cf6c6e2b9d49ee94341f10c0","author":{"name":"mulhern","email":"87548b8fc7ce9ff25c34053404638f744f7df56b@redhat.com"},"message":"Get rid of error return on BlockDevTier::Cache argument in pools\n\nJust return Ok in SimPool, but in StratPool push error down into Backstore.\n\nSigned-off-by: mulhern <amulhern@redhat.com>","distinct":true,"url":"https://api.github.com/repos/mulkieran/stratisd/commits/2bea60864bd72db1cf6c6e2b9d49ee94341f10c0"},{"sha":"cf0013f6f30d68edf8f9b67923686c9400d270ba","author":{"name":"mulhern","email":"87548b8fc7ce9ff25c34053404638f744f7df56b@redhat.com"},"message":"Make top-level device cache or linear\n\nIt will be one or the other, depending on whether there are cache devices\nor not.\n\nThis is a temporary commit. All unwrap()s and panic!()s must go away in\nfinal version.\n\nSigned-off-by: mulhern <amulhern@redhat.com>","distinct":true,"url":"https://api.github.com/repos/mulkieran/stratisd/commits/cf0013f6f30d68edf8f9b67923686c9400d270ba"},{"sha":"fb854bf9cf4bf429f90afa049c942bc0fe9a41c7","author":{"name":"mulhern","email":"87548b8fc7ce9ff25c34053404638f744f7df56b@redhat.com"},"message":"Add a new() method for the CacheTier\n\nIt is not yet in use.\n\nSigned-off-by: mulhern <amulhern@redhat.com>","distinct":true,"url":"https://api.github.com/repos/mulkieran/stratisd/commits/fb854bf9cf4bf429f90afa049c942bc0fe9a41c7"},{"sha":"59d2efa3664d14b056ff61431e1b8d181b0c8ae1","author":{"name":"mulhern","email":"87548b8fc7ce9ff25c34053404638f744f7df56b@redhat.com"},"message":"No longer use either for cache or linear device\n\nIt is too constraining, because we need to swap into dm_device something\nthat was made from it, but only after we have removed it.\nSo, use two Option fields and lots of logic.\n\nSigned-off-by: mulhern <amulhern@redhat.com>","distinct":true,"url":"https://api.github.com/repos/mulkieran/stratisd/commits/59d2efa3664d14b056ff61431e1b8d181b0c8ae1"},{"sha":"f4cfd17c37aaa99da4b16e7b284ce9df7dac9bb6","author":{"name":"mulhern","email":"87548b8fc7ce9ff25c34053404638f744f7df56b@redhat.com"},"message":"Create a CacheTier when cache dev is added if not there already\n\nThe idea is that when the user adds their first cachedev, they will know\nthat it is the first, and they will want the cache to start up.\n\nSigned-off-by: mulhern <amulhern@redhat.com>","distinct":true,"url":"https://api.github.com/repos/mulkieran/stratisd/commits/f4cfd17c37aaa99da4b16e7b284ce9df7dac9bb6"},{"sha":"7b3b38a9666c165313c4f1bb720181f03696144b","author":{"name":"mulhern","email":"87548b8fc7ce9ff25c34053404638f744f7df56b@redhat.com"},"message":"Have Backstore get pool uuid from its own DataTier\n\nThis is not intended to be a permanent solution to the problem of obtaining\nthe pool uuid when initializing cache blockdevs.\n\nSigned-off-by: mulhern <amulhern@redhat.com>","distinct":true,"url":"https://api.github.com/repos/mulkieran/stratisd/commits/7b3b38a9666c165313c4f1bb720181f03696144b"},{"sha":"3a25378eb8d1b1afb12a6f78c3320f7a820b9680","author":{"name":"mulhern","email":"87548b8fc7ce9ff25c34053404638f744f7df56b@redhat.com"},"message":"Get rid of dead_code clippy annotation\n\nIt is obsolete.\n\nSigned-off-by: mulhern <amulhern@redhat.com>","distinct":true,"url":"https://api.github.com/repos/mulkieran/stratisd/commits/3a25378eb8d1b1afb12a6f78c3320f7a820b9680"},{"sha":"cbdd3bc9ce50e02112b2b44d4d602984ed98b92e","author":{"name":"mulhern","email":"87548b8fc7ce9ff25c34053404638f744f7df56b@redhat.com"},"message":"Set the table on the cache sub-device using CacheDev::set_cache_table\n\nCacheDev::set_cache_table() was newly introduced in devicemapper version\n0.18.\n\nSigned-off-by: mulhern <amulhern@redhat.com>","distinct":true,"url":"https://api.github.com/repos/mulkieran/stratisd/commits/cbdd3bc9ce50e02112b2b44d4d602984ed98b92e"},{"sha":"c36539b8d4eb802e5f0806ee291a1d0fb9f651a6","author":{"name":"mulhern","email":"87548b8fc7ce9ff25c34053404638f744f7df56b@redhat.com"},"message":"Add some simple tests to exercise the D-Bus layer on AddCacheDevs\n\nSince the simulator engine doesn't do anything with cache devs, these\ntests are necessarily very broad.\n\nSigned-off-by: mulhern <amulhern@redhat.com>","distinct":true,"url":"https://api.github.com/repos/mulkieran/stratisd/commits/c36539b8d4eb802e5f0806ee291a1d0fb9f651a6"},{"sha":"8ebe759d1e37a1475a08e3e120cdde8061dd3abb","author":{"name":"mulhern","email":"87548b8fc7ce9ff25c34053404638f744f7df56b@redhat.com"},"message":"Add a basic test for adding cachedevs to the backstore\n\nIt just detects that the cachedevs were added and that the internal\nstate of the backstore has changed.\n\nSigned-off-by: mulhern <amulhern@redhat.com>","distinct":true,"url":"https://api.github.com/repos/mulkieran/stratisd/commits/8ebe759d1e37a1475a08e3e120cdde8061dd3abb"}]}
{"actor":{"display_login":"mulkieran"}}