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 Version / ansible instructions #6

Merged
merged 1 commit into from
May 3, 2024
Merged

Conversation

gigadjo
Copy link
Contributor

@gigadjo gigadjo commented May 2, 2024

Hello,

I modified readme file, to :

  • add version of ceph, reef seem to be needed on the control node for the ansible script to run. It's not default version on debian.
    (had problems running "ceph mon enable-msgr2" line 317 of ceph.yaml, it would freese deployment that errors)
  • added a link to ceph linux package install instructions
  • added a few instructions to guide those who wants to run this script toward real hardware.

Hope it helps,

@stgraber
Copy link
Member

stgraber commented May 3, 2024

  • add version of ceph, reef seem to be needed on the control node for the ansible script to run. It's not default version on debian.
    (had problems running "ceph mon enable-msgr2" line 317 of ceph.yaml, it would freese deployment that errors)

That doesn't seem to be true here, my deployment system is Debian 12 and things work fine here.
When things freeze it's most often than not because you've done two deploys in a row and didn't clean up the data in data/*/.

README.md Outdated
Comment on lines 16 to 19
Version required : reef

Install instructions : [Ceph Doc](https://docs.ceph.com/en/latest/install/get-packages/)

Copy link
Member

Choose a reason for hiding this comment

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

As mentioned, that's not actually required, I've been doing testing using 16.2 (stock from Debian 12) and heard it working even with 15.2 (Ubuntu 20.04).

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Oh, my bad !
I effectively tried to run once, it froze, i turned it off, destroyed everything with Terraform, re-ran, and it failed then !..
I did not took care of removing files generated in the local folders !

I'll try to add it somehow to the readme, if you think it is of some interest.. or maybe average users are a little bit carefull of what they do :-)

Have a nice day,

/joen

@gigadjo gigadjo closed this May 3, 2024
README.md Outdated
@@ -45,3 +49,92 @@ Run the Playbooks:
```
ansible-playbook deploy.yaml
```

## How to use in prod, for newbees :
Copy link
Member

Choose a reason for hiding this comment

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

Newbies

README.md Outdated

One can edit hosts.yaml.example to fit his needs :

First of all, use uuidgen to generate a unique UUID for your deployment, and remplace it in hosts.yaml.
Copy link
Member

Choose a reason for hiding this comment

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

Should probably mention that this is the ceph_fsid key.

Copy link
Member

Choose a reason for hiding this comment

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

There are seperate ovn_name and incus_name keys for the other two pieces.

@gigadjo gigadjo reopened this May 3, 2024
@gigadjo
Copy link
Contributor Author

gigadjo commented May 3, 2024

Hello, sorry, i'm new to contributing.. and to the actual Github logic :-)

I tried to follow what you asked, and added a few infos about "re-running" a deployment that failed.. :-)

Have a nice day,

/joen

@stgraber stgraber merged commit 40943c2 into lxc:main May 3, 2024
2 checks passed
@stgraber
Copy link
Member

stgraber commented May 3, 2024

I took your changes and did a bit of rewording and changing the format a bit, that should cover roughly the same information but be a bit easier to maintain.

@gigadjo
Copy link
Contributor Author

gigadjo commented May 3, 2024

Yeah i noticed it, and i must admit, it's best said your way than mine :-)

Played a bit with incus today, ended writing two inventory for coworkers to understand what i was playing with, it turned as an over-filled inventory file (based on yours) and a minimalistic one, intended to deploy incus as a 3 server cluster, with bare local storage and macvlan.

It's time to go home now, but i'll try to set something "inbetween", like Incus + OVN (with no Ceph) and Incus + Ceph (with no OVN).. and demo it aswell.

Maybe one day, my employer will see the light, and drop VMWare.. i'm actively working on it, and you're helping a lot in this :-)

Best regards,

/joen

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants