]> git.apps.os.sepia.ceph.com Git - ceph-ci.git/commit
Revert "dpdk: drop dpdk submodule"
authorDavid Galloway <dgallowa@redhat.com>
Thu, 30 Jan 2020 18:41:28 +0000 (13:41 -0500)
committerDavid Galloway <dgallowa@redhat.com>
Thu, 30 Jan 2020 18:41:28 +0000 (13:41 -0500)
commit2d3dc03878e8f7c4915243135b6fedef07b31717
tree6a2ea0813aca5ee9a6a71c6d03caf0a01d335a57
parent2954c607b7494e73dea059dc2454bdaf287f000a
Revert "dpdk: drop dpdk submodule"

This reverts commit cb8087dfac31b8490fefdfca28d389b7b9901ef8.

The seastar submodule's .gitmodules links to `../dpdk` which is no longer present after removing dpdk from ceph.git's .gitmodules.

```
<dwfreed> the ceph/seastar repo uses awful URLs for the submodules
<dwfreed> and those awful URLs are the real reason it's failing
<dwfreed> dgalloway: ^^^
<dwfreed> seastar's .gitmodules references repos in the parent directory, so that when it's checked out as a submodule of ceph, you don't download the repos twice (and git will probably also use references instead of duplicating the local .git); however, ceph doesn't have a submodule for dpdk anymore
<dwfreed> so seastar's referencing a dpdk repo that doesn't exist
<dgalloway> i think i follow.  so you're suggesting revert https://github.com/ceph/ceph/commit/cb8087dfac31b8490fefdfca28d389b7b9901ef8 ?
<dwfreed> yep
<dwfreed> that'd be one way to fix it
...
<joshd> dgalloway: I'd suggest revert for now, and let the crimson folks figure out the longer term fix when they're back
```

Signed-off-by: David Galloway <dgallowa@redhat.com>
.gitmodules