Review `MicroCeph` charm for listing

Hey, I’d like to propose Charm-MicroCeph for listing. It has been in use for some time as a sunbeam project and is developed and published by Openstack Charmers.

Please find relevant links required for listing below and please let me know if there are any questions or concerns :smiley:

Links

Charm Review PR

The upstream repositories has direct push disabled hence I’ve created a pull request (following steps here) from my fork of charm-microceph.

Charm Links

  1. Charmhub.io

  2. Github Repository

  3. Bugs

CI Links

  1. Lint Check

  2. Unit Tests

  3. Functional Tests

  4. Release Automation

Test Results

  1. Lint Test Result

  2. Unit Test Result

  3. Functional Test Result

  4. Release Automation CI Result

Documentation Links

  1. Usage Docs

  2. License

  3. Contribution Docs: Openstack Charm-Guide

  4. Communication:

Thanks @utkarshbhatthere for the review request! @ca-scribner can you please help with this review? You can go through and tick off the items on the checklist available here and post the result in this thread. This prior listing request could serve as an example of how to go about it. Please ping @review-coordinators for any questions.

Hi @utkarshbhatthere , @ca-scribner ,

I have doubts ref listing of this charm in the current form, please allow me to share them:

  1. The charm seems to be published only to edge so far - does it indicate it is work in progress? How about considering a release to stable first before switching it to listed
  2. The documentation is very minimal (almost a “one liner”) and no further links are provided. I have the impression that there can be much more around a Microceph charm to talk about and to refer to.

Hello @mcjaeger Thanks for the review, please allow me to respond to your concerns.

  1. Stable release: Charm MicroCeph is indeed only released to edge until now. There is some on-going feature work that needs to go in before we mark it stable. However, the charm (in its current state) works and is used for all sunbeam storage role deployments. I was not aware of stable release requirement for listing, that will take around 10 days of time.

  2. Documentation: We are maintaining relevant usage docs in the project README, I am in the process of moving relevant parts to a Discourse Article and it should reflect on charmhub soon.

hello,

documentation and links are mentioned in the review description being necessary, right now, the charm misses both on charmhub.

Ref. the formal requirement if the charm should be to stable. I am not sure about the formal requirement, but allow me to express again that it is not clear to me if the charm is ready for being used by others (which would be the anticipated consequence of listing) if you just have released to edge only so far.

What would be the work required to bring it to the stable channel? If the charms works reliably and works in all sunbeam storage role deployments, what prevents you form publishing a first release to the stable channel? How about making a release to stable first and then submit for review?

@mcjaeger Oh yeah, I was not trying to suggest listing before the 2 were resolved. Was just updating on their status. I will keep this review request live and ping back once the items are resolved ??

1 Like

can you take another look at this request please ? We have now released to reef/stable and the documentation state is much mature :smiley:

poke @ca-scribner, updated, can you please take another look ?

FYI, @ca-scribner’s calendar shows PTO for this week. Can the listing wait until mid next week? If not then I can pick it up instead.

I am sorry for missing this. It’s OK, will check it in on Monday.

hey thanks for your patience. Looks good to me!

  • Intended functionality - :white_check_mark:

  • charmhub page - :white_check_mark:

  • Source repository - :white_check_mark:

  • Coding conventions - :white_check_mark:

  • Release automation implementation - :white_check_mark:

  • Unit tests implementation - :white_check_mark:

  • Unit tests results - :white_check_mark:

  • Installation test implemented (could be part of the integration test) - :white_check_mark:

  • Installation test results - :white_check_mark:

  • Integration tests implemented - :white_check_mark:

  • Integration test results - :white_check_mark:

  • Documentation for usage - :white_check_mark:

  • Documentation for contributing - :white_check_mark:

  • Licensing statement - :white_check_mark:

2 Likes

Thanks @ca-scribner!

@odysseus-k This one should be good for listing!

1 Like

Hi!

@tony-meyer, this charm is now listed.

1 Like

Hey Emma,

I am still unable to see this charm when searching for it on the charmhub.io homepage. Output: No packages match this filter

Reference: Charmhub | The Open Operator Collection

Hi Utkarsh,

I’ve noted the above. The charm appears to be ‘listed’ but does not show up in Charmhub.io. Taking a look into this issue for you. I’ll update you when I’ve got a solution.

Thanks, Emma

1 Like

Hi @utkarshbhatthere !

Apologies for the belated response here. The reason your charm is not showing up on Charmhub.io is because the release that the charm is defaulted to is the latest/edge release. Charms will not show up unless they are defaulted to a channel that has a stable release.

Your two options would be to change the default track to reef (which I can do). That should fix the issue. Option two would be to release a revision to latest/stable.

Please let me know what you would like to do.

Thanks, Emma

1 Like

Hey, Thanks for the update, setting the default to reef would be perfect.

Thanks and Regards!

I have updated the default track to be reef, which I believe should solve the problem. Give it around 15 mn to take effect and then I can ensure that the charm is listed.

Doesn’t look like it’s appearing. Investigating …

Have you released the reef/stable revision?