Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ceph: improve building of dev images #512

Closed
wants to merge 1 commit into from
Closed

Conversation

epuertat
Copy link
Member

  • Set NVMEOF_GW for vstart to work with nvmeof.
  • Help debug ceph.repo issues.
  • Add new CEPH_CLUSTER_CEPH_REPO_BASEURL alternative to setting CEPH_BRANCH and CEPH_SHA.
  • Fix setting CEPH_CLUSTER_CEPH_REPO_BASEURL in Makefile not to override value if externally set (via .env).

- Set `NVMEOF_GW` for `vstart` to work with nvmeof.
- Help debug ceph.repo issues.
- Add new `CEPH_CLUSTER_CEPH_REPO_BASEURL` alternative to setting
  `CEPH_BRANCH` and `CEPH_SHA`.
- Fix setting `CEPH_CLUSTER_CEPH_REPO_BASEURL` in `Makefile` not to
  override value if externally set (via .env).

Signed-off-by: Ernesto Puerta <[email protected]>
@epuertat epuertat force-pushed the wip-ceph_build-devel branch from 7c3dc0c to 742bba9 Compare March 20, 2024 09:51
docker-compose.yaml Show resolved Hide resolved
.env Show resolved Hide resolved
@epuertat
Copy link
Member Author

BTW, @baum this change broke support for ceph-devel and other use cases of this make + docker-compose. I was planning to fork this repo and revert that commit.

@baum
Copy link
Collaborator

baum commented Mar 21, 2024

BTW, @baum this change broke support for ceph-devel and other use cases of this make + docker-compose. I was planning to fork this repo and revert that commit.

@epuertat up Makefile target no longer dispatches to docker-compose, due to requirement to send nvme gw create ceph monitor commands during gateway deployment. Any ideas on how to improve it? For ceph-devel README now recommends doing "docker-compose up nvmeof-devel" 😀 Please share your thoughts.

Copy link
Collaborator

@baum baum left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm 🖖

labels:
io.ceph.nvmeof:
environment:
TOUCHFILE: /tmp/ceph.touch
NVMEOF_GW: ${NVMEOF_IP_ADDRESS}:${NVMEOF_GW_PORT}
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

How is the NVMEOF_GW variable utilized? And what occurs in a multi-gateway deployment?

@baum baum closed this Sep 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants