Created attachment 192029 [details] databases/py-carbonate This brings databases/py-carbonate(https://github.com/graphite-project/carbonate). It is a set of utilities provided by the graphite-project that are very useful when you're operating carbon datastores. For instance should the number of machines in your cluster grow these utitilties can help you rebalance the cluster. It is also useful if you loose a node or accidentally delete all the metrics. It can help you sync that node. This port has three patches included: * files/patch-carbonate_cluster.py - Fixes a bug, but there is no new release yet so I backported this fix (https://github.com/graphite-project/carbonate/commit/ac50289a3114d4c47d84284ac24950218ed061db) * files/patch-carbonate_util.py - Look for the carbonate.conf file in /usr/local/etc instead, which is more freebsd-ish (default is /opt/graphite/conf/. * files/patch-carbonate_util.py - Patches the default root directory where to look for whisper files, databases/py-carbon for instance writes to /var/db/carbon/whisper by default. I've tried this port and it works, a bit slow, but still, it works.
Thanks Andreas. Can you please confirm QA passes?
Comment on attachment 192029 [details] databases/py-carbonate Maintainer approval doesn't apply to new (yet uncommited) ports
(In reply to Kubilay Kocak from comment #1) Yes! Poudriere QA passes with 10.3, 10.4, 11.1 and 12-CURRENT. portlint -AC is also fine. Functionality also verified with a very real usecase (the very reason I needed this tool). :)
I'll commit this now since you're so excited.
A few review items: /usr/local should never be hardcoded. In this case, since we cant pass it as a configuration to the build, use %%PREFIX%% in the patch, and then use REINPLACE_CMD to replace it with the current ${PREFIX} value at post-patch. [1] Either add a comment above the commented out LICENSE_FILE to explain it ('not yet packaged with distfile'), or leave the line out. Without a comment it looks accidental. Since the python package installs console scripts (things into localbase/bin) [2], make the port concurrent versions safe by using USE_PYTHON=concurrent, which will automatically version suffix all scripts and create (version-less) symlinks to point the versioned scriptnames. This isn't a major issue as the port currently only supports 2.7, but the port should be made concurrent safe anyway. Add comments to patches, in particular for ones that aren't self explanatory. In this case the patch that adds 'ROUTER_HASH_TYPE = None'. You (awesomely) explained (most?) of them in your initial comment 0, add those comments and references to the top of all the patches themselves. [1] See: 4.4.3. Simple Automatic Replacements in Porters Handbook [2] https://github.com/graphite-project/carbonate/blob/master/setup.py#L37
Created attachment 192050 [details] Add concurrent to USES_PYTHON=, use ${REINPLACE_CMD} Thank you for your comments. This new diff removes two of the patches, and instead uses ${REINPLACE_CMD} to do two simple path changes to be more FreeBSD like. It also comments on why the patch was made, with a reference to a thread on github. Lastly this port now adds concurrent to USES_PYTHON.
According to this: https://github.com/graphite-project/carbonate/pull/97 we might want to wait a day or two to commit this port since new version might be released soon as well as maybe getting tests which was requested by Kubilay
(In reply to Andreas Andersson from comment #7) Will update my patch in one or two days. :) https://github.com/graphite-project/carbonate/issues/96
Created attachment 192202 [details] Update to 1.1.3 This updates to 1.1.3, thus I removed the remaining patch and the files/ directory since new release has the cluster.py fix. Poudriere QA fine on 10.3, 10.4, 11.1 and 12-CURRENT. portlint -AC also fine.
Created attachment 205037 [details] py-carbonate-1.1.5.patch (In reply to Andreas Andersson from comment #9) Hi Andreas, at first: Sorry for the huge delay! Somehow this PR totally went under. Meanwhile the latest release is 1.1.5 that has support for Python 3, that can be enabled once the dependent port databases/py-carbon is enabled for Python 3. Attached you'll find an updated patch based on your initial patch with the following additions: - Added LICENSE_FILE (upstream now packages that file) - Improved pkg-descr a bit to give the users some more info about the port If you're still interested to get this port into the ports tree and the 1.1.5 release works for you then I'm happy to commit it asap.
Closing this PR with "Feedback Timeout" as it seems that there's no more interest to bring this port into the ports tree. Andreas, feel free to re-open this PR if you still want databases/py-carbonate into the Ports tree.