Skip to content

Use Zope 6.2 - #1108

Merged
gforcada merged 1 commit into
6.2from
maurits-zope-62
Aug 6, 2026
Merged

Use Zope 6.2#1108
gforcada merged 1 commit into
6.2from
maurits-zope-62

Conversation

@mauritsvanrees

Copy link
Copy Markdown
Member

And latest pip.

@mauritsvanrees

Copy link
Copy Markdown
Member Author

@jenkins-plone-org please run jobs

@gforcada

gforcada commented Aug 5, 2026

Copy link
Copy Markdown
Member

Where is Zope 6.2 being used? 🤔 on the diff I don't see any reference to it 🙃

@mauritsvanrees

Copy link
Copy Markdown
Member Author

Didn't I save the file? Oops! :-D
Fixed now, and force pushed. Thanks!

I canceled the Jenkins jobs. Try again:

@jenkins-plone-org please run jobs

@mauritsvanrees

Copy link
Copy Markdown
Member Author

Products.PluggableAuthService needs test fixes, related to the XML-RPC change. I am having a look.

@mauritsvanrees

Copy link
Copy Markdown
Member Author

@jenkins-plone-org please run jobs

@mauritsvanrees

mauritsvanrees commented Aug 5, 2026

Copy link
Copy Markdown
Member Author

Robot tests fail with:

AssertionError: No keyword with name 'Enable autologin as' found.

No idea why yet.

Or maybe this is because of what is printed right after:

[ ERROR ] Error in file '/home/runner/work/buildout.coredev/buildout.coredev/.venv/lib/python3.14/site-packages/plone/schemaeditor/tests/robot/test_fields.robot' on line 5: 
Getting keyword names from library 'Remote' failed:
Calling dynamic method 'get_keyword_names' failed:
Connecting remote server at http://localhost:41083/plone/RobotRemote failed:
<ProtocolError for localhost:41083/plone/RobotRemote: 400 Bad Request>

I am retrying for good measure.

@davisagli

Copy link
Copy Markdown
Member

@mauritsvanrees The communication with the robot server uses xmlrpc, so that makes sense. The new Zope setting probably needs to be explicitly enabled in plone.app.robotframework

@mauritsvanrees

Copy link
Copy Markdown
Member Author

Wow, I did not know that plone.app.robotframework used XML-RPC. But you are right.

I am trying something, but so far it is tricky.

@mauritsvanrees

Copy link
Copy Markdown
Member Author

Okay, I have a small PR for plone.testing that should fix it:
plone/plone.testing#137

I am checking that branch out in the current PR.

@jenkins-plone-org please run jobs

mister-roboto pushed a commit that referenced this pull request Aug 6, 2026
Branch: refs/heads/master
Date: 2026-08-06T02:03:10+02:00
Author: Maurits van Rees (mauritsvanrees) <maurits@vanrees.org>
Commit: plone/plone.testing@8179a29

Explicitly enable XML-RPC in the ``WSGIServer`` layer.

Zope 6.2 by default disables this.
But this is used in the ``WSGI_SERVER_FIXTURE`` that most robot tests (and acceptance tests) use.
See #1108 (comment) and further.

Files changed:
A news/+0ae076a4.feature.rst
M src/plone/testing/zope.py
Repository: plone.testing

Branch: refs/heads/master
Date: 2026-08-06T14:14:56+02:00
Author: Maurits van Rees (mauritsvanrees) <maurits@vanrees.org>
Commit: plone/plone.testing@85d6850

Merge pull request #137 from plone/maurits-enable-xmlrpc

Explicitly enable XML-RPC in the ``WSGIServer`` layer.

Files changed:
A news/+0ae076a4.feature.rst
M src/plone/testing/zope.py
mister-roboto pushed a commit that referenced this pull request Aug 6, 2026
Branch: refs/heads/master
Date: 2026-08-06T02:03:10+02:00
Author: Maurits van Rees (mauritsvanrees) <maurits@vanrees.org>
Commit: plone/plone.testing@8179a29

Explicitly enable XML-RPC in the ``WSGIServer`` layer.

Zope 6.2 by default disables this.
But this is used in the ``WSGI_SERVER_FIXTURE`` that most robot tests (and acceptance tests) use.
See #1108 (comment) and further.

Files changed:
A news/+0ae076a4.feature.rst
M src/plone/testing/zope.py
Repository: plone.testing

Branch: refs/heads/master
Date: 2026-08-06T14:14:56+02:00
Author: Maurits van Rees (mauritsvanrees) <maurits@vanrees.org>
Commit: plone/plone.testing@85d6850

Merge pull request #137 from plone/maurits-enable-xmlrpc

Explicitly enable XML-RPC in the ``WSGIServer`` layer.

Files changed:
A news/+0ae076a4.feature.rst
M src/plone/testing/zope.py
And then we need `Products.PluggableAuthService` 4.2.2.
And we upgrade `pip` to 26.2.1.
@mauritsvanrees

Copy link
Copy Markdown
Member Author

I have merged my plone.testing PR and have updated the Products.PluggableAuthService tests and made a release of that one.
I have rebased and force-pushed so we have one commit again.

@jenkins-plone-org please run jobs

@mauritsvanrees

Copy link
Copy Markdown
Member Author

Now all is green.

@gforcada
gforcada merged commit abc0920 into 6.2 Aug 6, 2026
40 checks passed
@gforcada
gforcada deleted the maurits-zope-62 branch August 6, 2026 14:42
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.

3 participants