URL: https://github.com/freeipa/freeipa/pull/372
Title: #372: Restore IPA 3.0 compatibility of copy-schema-to-ca.py
tiran commented:
"""
I have updated the ticket
https://fedorahosted.org/freeipa/ticket/6540#comment:5 with the result of this
discussion. I'm going to close the PR. Let's start
URL: https://github.com/freeipa/freeipa/pull/372
Title: #372: Restore IPA 3.0 compatibility of copy-schema-to-ca.py
mbasti-rh commented:
"""
@tiran +1, but first this has to be generally approved :) topic for meeting
maybe (today?)
"""
See the full comment at
URL: https://github.com/freeipa/freeipa/pull/372
Title: #372: Restore IPA 3.0 compatibility of copy-schema-to-ca.py
tiran commented:
"""
Or we just grab a working and tested version from an old release.
"""
See the full comment at
URL: https://github.com/freeipa/freeipa/pull/372
Title: #372: Restore IPA 3.0 compatibility of copy-schema-to-ca.py
stlaz commented:
"""
+1, we need to fix the script first, though.
"""
See the full comment at
https://github.com/freeipa/freeipa/pull/372#issuecomment-273108618
--
Manage your
URL: https://github.com/freeipa/freeipa/pull/372
Title: #372: Restore IPA 3.0 compatibility of copy-schema-to-ca.py
tiran commented:
"""
So with *separate script* you meant a separate downloadable version of the
script. Got it! :)
It seems we have a consent. @mbasti-rh I second your proposal
URL: https://github.com/freeipa/freeipa/pull/372
Title: #372: Restore IPA 3.0 compatibility of copy-schema-to-ca.py
mbasti-rh commented:
"""
I proposed 2 ideas:
- move it to ipa-3-3 branch (or)
- extract that script from freeipa repo and allow to download that script from
freeipa.org (and
URL: https://github.com/freeipa/freeipa/pull/372
Title: #372: Restore IPA 3.0 compatibility of copy-schema-to-ca.py
martbab commented:
"""
IIRC @mbasti-rh proposed to maintain the script separately and serve it to
users via external repo or something but the idea was rejected.
"""
See the
URL: https://github.com/freeipa/freeipa/pull/372
Title: #372: Restore IPA 3.0 compatibility of copy-schema-to-ca.py
tiran commented:
"""
How about we remove the file entirely and just post it on the wiki or something?
"""
See the full comment at
URL: https://github.com/freeipa/freeipa/pull/372
Title: #372: Restore IPA 3.0 compatibility of copy-schema-to-ca.py
stlaz commented:
"""
+1, that was actually my original point. Just revert the change done to the
file in
URL: https://github.com/freeipa/freeipa/pull/372
Title: #372: Restore IPA 3.0 compatibility of copy-schema-to-ca.py
HonzaCholasta commented:
"""
I agree with @mbasti-rh. IMO we should remove all 4.0+ specific code from the
script, add a version check at the beginning and disable all failing
URL: https://github.com/freeipa/freeipa/pull/372
Title: #372: Restore IPA 3.0 compatibility of copy-schema-to-ca.py
mbasti-rh commented:
"""
this script must work only with IPA3.x, so I wouldn't add there anything from
4.4/master code. As I pointed out several times I don't think that this
URL: https://github.com/freeipa/freeipa/pull/372
Title: #372: Restore IPA 3.0 compatibility of copy-schema-to-ca.py
tiran commented:
"""
```SHARE_DIR``` is no longer available. I had to find another approach. The
approach ```import else use well-known constants``` is safe and will not break
URL: https://github.com/freeipa/freeipa/pull/372
Title: #372: Restore IPA 3.0 compatibility of copy-schema-to-ca.py
stlaz commented:
"""
Is there a reason not to stick with the original `ipautil.SHARE_DIR` and
without setting `confdir`? This script won't be run on servers that either need
13 matches
Mail list logo