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

Created S3 test how-to #306

Closed
wants to merge 61 commits into from
Closed

Created S3 test how-to #306

wants to merge 61 commits into from

Conversation

cgsamp
Copy link

@cgsamp cgsamp commented Feb 17, 2024

Created a new how-to describing the process I used to test the Ceph S3 service using Microceph. I am not familiar with rst documentation but attempted to copy format from an example, and used markdown. I used a ubuntu server 22 virtual machine to deploy microceph, and used s3cmd on my local macbook. I have not tested installing and using s3cmd on Windows.

I am happy for the maintainers to edit in any way, and will respond to any suggestions or requests for revision.

Thank you!

lmlg and others added 11 commits February 5, 2024 21:57
This PR implements the setter and getter for log levels as used
by the microceph daemon. It needs to jump through some hoops, as
the lxd libraries don't expose this functionality themselves.

Signed-off-by: Luciano Lo Giudice <luciano.logiudice@canonical.com>
Signed-off-by: Luciano Lo Giudice <luciano.logiudice@canonical.com>
Signed-off-by: Luciano Lo Giudice <luciano.logiudice@canonical.com>
Signed-off-by: Luciano Lo Giudice <luciano.logiudice@canonical.com>
Signed-off-by: Luciano Lo Giudice <luciano.logiudice@canonical.com>
Signed-off-by: Matthias Kesler <krombel@krombel.de>
@cgsamp
Copy link
Author

cgsamp commented Feb 18, 2024

I have tried to format the JSON user config appropriately, but I can't find the spec for the json linter and don't want to mess with your docs/.sphinx/requirements.txt. It seems to be guiding me to a flat one-indent-level format, which isn't how I prefer to format nested json.

Implement the operations for getting and setting log-level
@sabaini
Copy link
Collaborator

sabaini commented Feb 20, 2024

Hey @cgsamp thanks for that nice howto. However we would need a contributor licence and have the commit signed before we can merge I'm afraid. Cf. https://ubuntu.com/legal/contributors for details.

…param

Fix canonical#303: Introduce flag --microceph-ip for cluster bootstrap
@cgsamp
Copy link
Author

cgsamp commented Feb 20, 2024 via email

lmlg and others added 14 commits February 20, 2024 14:50
Signed-off-by: Luciano Lo Giudice <luciano.logiudice@canonical.com>
Signed-off-by: Luciano Lo Giudice <luciano.logiudice@canonical.com>
Signed-off-by: Luciano Lo Giudice <luciano.logiudice@canonical.com>
Signed-off-by: Luciano Lo Giudice <luciano.logiudice@canonical.com>
Signed-off-by: Luciano Lo Giudice <luciano.logiudice@canonical.com>
Signed-off-by: Luciano Lo Giudice <luciano.logiudice@canonical.com>
This PR aims to fix the removal test to take into consideration
that a node that is being removed may be temporarily reported
as 'out of quorum' instead of completely out.

Signed-off-by: Luciano Lo Giudice <luciano.logiudice@canonical.com>
Signed-off-by: Peter Sabaini <peter.sabaini@canonical.com>
Also test listing disks

Signed-off-by: Peter Sabaini <peter.sabaini@canonical.com>
Signed-off-by: Utkarsh Bhatt <utkarsh.bhatt@canonical.com>
Signed-off-by: Simon Deziel <simon.deziel@canonical.com>
Signed-off-by: Simon Deziel <simon.deziel@canonical.com>
Signed-off-by: Simon Deziel <simon.deziel@canonical.com>
simondeziel and others added 19 commits February 21, 2024 17:36
Signed-off-by: Simon Deziel <simon.deziel@canonical.com>
Signed-off-by: Simon Deziel <simon.deziel@canonical.com>
Signed-off-by: Simon Deziel <simon.deziel@canonical.com>
Signed-off-by: Simon Deziel <simon.deziel@canonical.com>
Signed-off-by: Simon Deziel <simon.deziel@canonical.com>
Signed-off-by: Simon Deziel <simon.deziel@canonical.com>
Signed-off-by: Simon Deziel <simon.deziel@canonical.com>
Signed-off-by: Utkarsh Bhatt <utkarsh.bhatt@canonical.com>
Signed-off-by: Utkarsh Bhatt <utkarsh.bhatt@canonical.com>
Signed-off-by: Luciano Lo Giudice <luciano.logiudice@canonical.com>
Tricks and enhancements from LXD and MicroCloud
Signed-off-by: Luciano Lo Giudice <luciano.logiudice@canonical.com>
Signed-off-by: Luciano Lo Giudice <luciano.logiudice@canonical.com>
Signed-off-by: Luciano Lo Giudice <luciano.logiudice@canonical.com>
Signed-off-by: Luciano Lo Giudice <luciano.logiudice@canonical.com>
Make it a direct dependency (otherwise the "go get" during build
dirties the worktree)

Signed-off-by: Peter Sabaini <peter.sabaini@canonical.com>
…ompatFix

Make Batch Disk addition API compatible with older clients.
Document the log manipulation facilities.
@UtkarshBhatthere
Copy link
Contributor

@cgsamp the prompt says PR or contact Please add the Canonical Project Manager or contact.

@sabaini
Copy link
Collaborator

sabaini commented Feb 26, 2024

Hi Peter, I did fill that out over the weekend, but didn't identify anyone as the "Canonical Project Manager." Should I complete it again and list you?

Hi @cgsamp , no I don't believe that's necessary. The test seems to be failing b/c it can't find the account you listed (c***@g***.com) on launchpad afaict

UtkarshBhatthere and others added 2 commits February 26, 2024 14:36
Signed-off-by: Utkarsh Bhatt <utkarsh.bhatt@canonical.com>
@cgsamp
Copy link
Author

cgsamp commented Feb 26, 2024

I completed the form again. I believe it is now correct.

@sabaini
Copy link
Collaborator

sabaini commented Feb 27, 2024

Hey @cgsamp , the CLA check succeeds now, nice!

We have 3 failing checks left over:

  • unit test: this is an unrelated test env issue
  • Commits signed off: we'd need your commits to be signed please -- ideally squash the commits into one and sign that
  • The spell checker tripped up a few times, I've pasted the results here. There are a number of words the spellchecker doesn't recognize, these would need to be added to the wordlist in
    docs/.custom_wordlist.txt

cheers!

@cgsamp
Copy link
Author

cgsamp commented Feb 27, 2024

Closing this one and creating a simpler version.

@cgsamp cgsamp closed this Feb 27, 2024
@cgsamp cgsamp deleted the cgsamp branch February 27, 2024 21:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants