[Bug 1441843] Review Request: linchpin - Ansible based multicloud orchestrator

https://bugzilla.redhat.com/show_bug.cgi?id=1441843 --- Comment #42 from Jonathan Dieter --- Man, I'm sorry. This is really frustrating, and I totally understand why you're throwing in the towel. :( -- You are receiving this mail because: You are on the CC list for the bug. You are always

[Bug 1441843] Review Request: linchpin - Ansible based multicloud orchestrator

https://bugzilla.redhat.com/show_bug.cgi?id=1441843 greg.helli...@gmail.com changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution|---

[Bug 1441843] Review Request: linchpin - Ansible based multicloud orchestrator

https://bugzilla.redhat.com/show_bug.cgi?id=1441843 --- Comment #40 from Jonathan Dieter --- They're coming from requirements.txt. See https://docs.fedoraproject.org/en-US/packaging-guidelines/Python/#_automatically_generated_dependencies -- You are receiving this mail because: You are on

[Bug 1441843] Review Request: linchpin - Ansible based multicloud orchestrator

https://bugzilla.redhat.com/show_bug.cgi?id=1441843 --- Comment #39 from greg.helli...@gmail.com --- Weird. I'm definitely not generating those narrow restrictions of versions. Any idea where they might be coming from? -- You are receiving this mail because: You are on the CC list for the

[Bug 1441843] Review Request: linchpin - Ansible based multicloud orchestrator

https://bugzilla.redhat.com/show_bug.cgi?id=1441843 --- Comment #38 from Jonathan Dieter --- I'd really like to see this move forward, but I can't even install it right now, in either Rawhide or F32: sudo /usr/bin/dnf --installroot /var/lib/mock/fedora-rawhide-x86_64/root/ --releasever 33

[Bug 1441843] Review Request: linchpin - Ansible based multicloud orchestrator

https://bugzilla.redhat.com/show_bug.cgi?id=1441843 --- Comment #37 from greg.helli...@gmail.com --- Updated this to latest version, which now fully supports Python 3 and drops Python 2 support. Scratch build: https://koji.fedoraproject.org/koji/taskinfo?taskID=43882055 SRPM:

[Bug 1441843] Review Request: linchpin - Ansible based multicloud orchestrator

https://bugzilla.redhat.com/show_bug.cgi?id=1441843 Petr Pisar changed: What|Removed |Added CC||ppi...@redhat.com Flags|

[Bug 1441843] Review Request: linchpin - Ansible based multicloud orchestrator

https://bugzilla.redhat.com/show_bug.cgi?id=1441843 greg.helli...@gmail.com changed: What|Removed |Added Depends On||1432701 Referenced Bugs:

[Bug 1441843] Review Request: linchpin - Ansible based multicloud orchestrator

https://bugzilla.redhat.com/show_bug.cgi?id=1441843 --- Comment #36 from greg.helli...@gmail.com --- I am, but the 1.7 series (the latest release of linchpin) requires Shade >= 1.30.0. The latest version packaged for Rawhide is 1.27. -- You are receiving this mail because: You are on the CC

[Bug 1441843] Review Request: linchpin - Ansible based multicloud orchestrator

https://bugzilla.redhat.com/show_bug.cgi?id=1441843 --- Comment #35 from Jonathan Dieter --- Greg, last I checked, we were almost finished with this. Are you still interested in moving forward with this? -- You are receiving this mail because: You are on the CC list for the bug. You are

[Bug 1441843] Review Request: linchpin - Ansible based multicloud orchestrator

https://bugzilla.redhat.com/show_bug.cgi?id=1441843 --- Comment #34 from Jonathan Dieter --- (In reply to greg.hellings from comment #33) > > The issues that still need to be resolved: > > > > [!]: License field in the package spec file matches the actual license. > > License should be

[Bug 1441843] Review Request: linchpin - Ansible based multicloud orchestrator

https://bugzilla.redhat.com/show_bug.cgi?id=1441843 --- Comment #33 from greg.helli...@gmail.com --- > The issues that still need to be resolved: > > [!]: License field in the package spec file matches the actual license. > License should be GPLv3+ and BSD I'm not sure why you're

[Bug 1441843] Review Request: linchpin - Ansible based multicloud orchestrator

https://bugzilla.redhat.com/show_bug.cgi?id=1441843 --- Comment #32 from Jonathan Dieter --- The issues that still need to be resolved: [!]: License field in the package spec file matches the actual license. License should be GPLv3+ and BSD [!]: Requires correct, justified where

[Bug 1441843] Review Request: linchpin - Ansible based multicloud orchestrator

https://bugzilla.redhat.com/show_bug.cgi?id=1441843 --- Comment #31 from Jonathan Dieter --- Package Review == Legend: [x] = Pass, [!] = Fail, [-] = Not applicable, [?] = Not evaluated = MUST items = Issues: [!]: License field in the package spec file matches the actual

[Bug 1441843] Review Request: linchpin - Ansible based multicloud orchestrator

https://bugzilla.redhat.com/show_bug.cgi?id=1441843 --- Comment #30 from greg.helli...@gmail.com --- Good catch on the docs/doc difference. I've updated the name of that and fixed its depends. Updated versions of the files pushed: Spec:

[Bug 1441843] Review Request: linchpin - Ansible based multicloud orchestrator

https://bugzilla.redhat.com/show_bug.cgi?id=1441843 --- Comment #29 from Jonathan Dieter --- Nicely done, but the requires for the -docs is wrong. linchpin is noarch, so it should be Requires: %{name} = %{version}-%{release}. As it is, I'm unable to actually install linchpin-docs. Also, not

[Bug 1441843] Review Request: linchpin - Ansible based multicloud orchestrator

https://bugzilla.redhat.com/show_bug.cgi?id=1441843 --- Comment #28 from greg.helli...@gmail.com --- Upstream release 1.6.4 today which fixes the doc build issue. I've uploaded an SRPM that takes advantage of that to include a docs subpackage as well as adding the README.rst to as a %doc

[Bug 1441843] Review Request: linchpin - Ansible based multicloud orchestrator

https://bugzilla.redhat.com/show_bug.cgi?id=1441843 --- Comment #27 from Jonathan Dieter --- (In reply to greg.hellings from comment #26) > The docs are to build the readthedocs page. They don't build in Python 3 due > to issues with imports. I have reported this upstream as >

[Bug 1441843] Review Request: linchpin - Ansible based multicloud orchestrator

https://bugzilla.redhat.com/show_bug.cgi?id=1441843 --- Comment #26 from greg.helli...@gmail.com --- The docs are to build the readthedocs page. They don't build in Python 3 due to issues with imports. I have reported this upstream as https://github.com/CentOS-PaaS-SIG/linchpin/issues/701 I've

[Bug 1441843] Review Request: linchpin - Ansible based multicloud orchestrator

https://bugzilla.redhat.com/show_bug.cgi?id=1441843 --- Comment #25 from Jonathan Dieter --- Ok, there still a few issues. First off, it looks like there's documentation in source, but no documentation seems to be installed as part of the package. Am I missing something? If not, can we

[Bug 1441843] Review Request: linchpin - Ansible based multicloud orchestrator

https://bugzilla.redhat.com/show_bug.cgi?id=1441843 greg.helli...@gmail.com changed: What|Removed |Added Flags|needinfo?(greg.hellings@gma | |il.com)

[Bug 1441843] Review Request: linchpin - Ansible based multicloud orchestrator

https://bugzilla.redhat.com/show_bug.cgi?id=1441843 Jonathan Dieter changed: What|Removed |Added CC||greg.helli...@gmail.com

[Bug 1441843] Review Request: linchpin - Ansible based multicloud orchestrator

https://bugzilla.redhat.com/show_bug.cgi?id=1441843 --- Comment #23 from Jonathan Dieter --- (In reply to Jonathan Dieter from comment #22) > (In reply to Jonathan Dieter from comment #21) > > Great, thanks. > > > > rpmlint is complaining about a number of empty hidden files in the RPM. Do >

[Bug 1441843] Review Request: linchpin - Ansible based multicloud orchestrator

https://bugzilla.redhat.com/show_bug.cgi?id=1441843 --- Comment #22 from Jonathan Dieter --- (In reply to Jonathan Dieter from comment #21) > Great, thanks. > > rpmlint is complaining about a number of empty hidden files in the RPM. Do > you know why they're there? If they're actually there

[Bug 1441843] Review Request: linchpin - Ansible based multicloud orchestrator

https://bugzilla.redhat.com/show_bug.cgi?id=1441843 --- Comment #21 from Jonathan Dieter --- Great, thanks. rpmlint is complaining about a number of empty hidden files in the RPM. Do you know why they're there? If they're actually there for a reason, then we'll ignore rpmlint, but if

[Bug 1441843] Review Request: linchpin - Ansible based multicloud orchestrator

https://bugzilla.redhat.com/show_bug.cgi?id=1441843 --- Comment #20 from greg.helli...@gmail.com --- Oh, yeah. I push them both every time, but the URL didn't change: Spec: https://fedorapeople.org/~greghellings/linchpin/linchpin.spec SRPM:

[Bug 1441843] Review Request: linchpin - Ansible based multicloud orchestrator

https://bugzilla.redhat.com/show_bug.cgi?id=1441843 --- Comment #19 from Jonathan Dieter --- Great. Do you mind posting the updated spec too? It makes it easier for fedora-review to run. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified

[Bug 1441843] Review Request: linchpin - Ansible based multicloud orchestrator

https://bugzilla.redhat.com/show_bug.cgi?id=1441843 --- Comment #18 from greg.helli...@gmail.com --- New SRPM: https://fedorapeople.org/~greghellings/linchpin/linchpin-1.6.1-1.fc30.src.rpm I updated all Python dependencies to Python 3. There were some dependency version problems in Rawhide

[Bug 1441843] Review Request: linchpin - Ansible based multicloud orchestrator

https://bugzilla.redhat.com/show_bug.cgi?id=1441843 --- Comment #17 from greg.helli...@gmail.com --- Ah, Rawhide Ansible package has shifted to Python 3, thus invalidating all of the Python 2 BRs. I'll update as appropriate and post a new SRPM in a few minutes. -- You are receiving this mail

[Bug 1441843] Review Request: linchpin - Ansible based multicloud orchestrator

https://bugzilla.redhat.com/show_bug.cgi?id=1441843 --- Comment #16 from Jonathan Dieter --- Created attachment 1481259 --> https://bugzilla.redhat.com/attachment.cgi?id=1481259=edit Build log Just tried to build this in mock and got a failure. It looks like it's trying to download a

[Bug 1441843] Review Request: linchpin - Ansible based multicloud orchestrator

https://bugzilla.redhat.com/show_bug.cgi?id=1441843 --- Comment #15 from Jonathan Dieter --- Yeah, I'll go ahead and do this. Sorry about the delay in getting back to you. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to

[Bug 1441843] Review Request: linchpin - Ansible based multicloud orchestrator

https://bugzilla.redhat.com/show_bug.cgi?id=1441843 --- Comment #14 from greg.helli...@gmail.com --- Jonathan, any chance you could look at this or would you rather let someone else tackle it? -- You are receiving this mail because: You are on the CC list for the bug. You are always notified

[Bug 1441843] Review Request: linchpin - Ansible based multicloud orchestrator

https://bugzilla.redhat.com/show_bug.cgi?id=1441843 --- Comment #13 from greg.helli...@gmail.com --- OK, giving this another go: Spec file: https://fedorapeople.org/~greghellings/linchpin/linchpin.spec SRPM: https://fedorapeople.org/~greghellings/linchpin/linchpin-1.5.4-1.fc28.src.rpm The

[Bug 1441843] Review Request: linchpin - Ansible based multicloud orchestrator

https://bugzilla.redhat.com/show_bug.cgi?id=1441843 --- Comment #12 from greg.helli...@gmail.com --- OK, I guess I better do this. Upstream started asking again if I was going to finish it up. Stay tuned, Friday is my designated Fedora Package day for the week. -- You are receiving this mail

[Bug 1441843] Review Request: linchpin - Ansible based multicloud orchestrator

https://bugzilla.redhat.com/show_bug.cgi?id=1441843 --- Comment #11 from Jonathan Dieter --- Any news on this? If not, I'll go ahead and remove myself from this and leave the bug as NEW. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified

[Bug 1441843] Review Request: linchpin - Ansible based multicloud orchestrator

https://bugzilla.redhat.com/show_bug.cgi?id=1441843 --- Comment #10 from Jonathan Dieter --- Yeah, sounds good. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component

[Bug 1441843] Review Request: linchpin - Ansible based multicloud orchestrator

https://bugzilla.redhat.com/show_bug.cgi?id=1441843 --- Comment #9 from greg.helli...@gmail.com --- It's a Bash completion script, so it's not intended to be run at all, rather sourced. It looks like that should be placed in /etc/bash_completion.d/ rather than in /usr/bin. I'll update to have

[Bug 1441843] Review Request: linchpin - Ansible based multicloud orchestrator

https://bugzilla.redhat.com/show_bug.cgi?id=1441843 --- Comment #8 from Jonathan Dieter --- Yep, that would be the one. If it's a script that's meant to run by linchpin (as opposed to the user), then it should probably go in /usr/libexec/linchpin. As for whether or not it

[Bug 1441843] Review Request: linchpin - Ansible based multicloud orchestrator

https://bugzilla.redhat.com/show_bug.cgi?id=1441843 --- Comment #7 from greg.helli...@gmail.com --- OK, the shebang and spurious file permissions are easy enough to fix. I'm curious which executable you think should have a shebang. Is it linchpin_complete.sh? Those completion scripts are not

[Bug 1441843] Review Request: linchpin - Ansible based multicloud orchestrator

https://bugzilla.redhat.com/show_bug.cgi?id=1441843 --- Comment #6 from greg.helli...@gmail.com --- Agreed. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component

[Bug 1441843] Review Request: linchpin - Ansible based multicloud orchestrator

https://bugzilla.redhat.com/show_bug.cgi?id=1441843 --- Comment #5 from Jonathan Dieter --- Ok, if it's all right with you, then, I'll go ahead and do the review once you've published the 1.0 package. -- You are receiving this mail because: You are on the CC list for the

[Bug 1441843] Review Request: linchpin - Ansible based multicloud orchestrator

https://bugzilla.redhat.com/show_bug.cgi?id=1441843 --- Comment #4 from greg.helli...@gmail.com --- I'll update the shebang/permissions issues, including sending a PR upstream. As for the packages, this issue has been addressed in upstream and will be released in their 1.0 milestone (due out

[Bug 1441843] Review Request: linchpin - Ansible based multicloud orchestrator

https://bugzilla.redhat.com/show_bug.cgi?id=1441843 --- Comment #3 from Jonathan Dieter --- Created attachment 1276746 --> https://bugzilla.redhat.com/attachment.cgi?id=1276746=edit rpmlint output Ok, we've got the dependencies in Fedora, so let's do this. There are a

[Bug 1441843] Review Request: linchpin - Ansible based multicloud orchestrator

https://bugzilla.redhat.com/show_bug.cgi?id=1441843 Bug 1441843 depends on bug 1441841, which changed state. Bug 1441841 Summary: Review Request: python-camel - Python serialization for adults https://bugzilla.redhat.com/show_bug.cgi?id=1441841 What|Removed

[Bug 1441843] Review Request: linchpin - Ansible based multicloud orchestrator

https://bugzilla.redhat.com/show_bug.cgi?id=1441843 Bug 1441843 depends on bug 1441842, which changed state. Bug 1441842 Summary: Review Request: python-yamlordereddictloader - YAML loader for PyYAML that maintains key order https://bugzilla.redhat.com/show_bug.cgi?id=1441842 What

[Bug 1441843] Review Request: linchpin - Ansible based multicloud orchestrator

https://bugzilla.redhat.com/show_bug.cgi?id=1441843 Jonathan Dieter changed: What|Removed |Added Status|NEW |ASSIGNED

[Bug 1441843] Review Request: linchpin - Ansible based multicloud orchestrator

https://bugzilla.redhat.com/show_bug.cgi?id=1441843 greg.helli...@gmail.com changed: What|Removed |Added Depends On||1441841 (python-camel),

[Bug 1441843] Review Request: linchpin - Ansible based multicloud orchestrator

https://bugzilla.redhat.com/show_bug.cgi?id=1441843 --- Comment #1 from greg.helli...@gmail.com --- n.b. If you're testing basic functionality, this program only runs on Fedora 26+ due to its dependency versions. -- You are receiving this mail because: You are on the CC list for the bug. You