Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[DO NOT MERGE] test for twisted trunk #5739

Closed
wants to merge 3 commits into from
Closed

[DO NOT MERGE] test for twisted trunk #5739

wants to merge 3 commits into from

Conversation

rodrigc
Copy link
Contributor

@rodrigc rodrigc commented Jan 3, 2021

Follow on to #5591

@codecov
Copy link

codecov bot commented Jan 3, 2021

Codecov Report

Merging #5739 (0be5fc8) into master (500d6d6) will decrease coverage by 0.02%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #5739      +/-   ##
==========================================
- Coverage   91.72%   91.69%   -0.03%     
==========================================
  Files         340      340              
  Lines       36866    36866              
==========================================
- Hits        33814    33804      -10     
- Misses       3052     3062      +10     
Impacted Files Coverage Δ
master/buildbot/scripts/cleanupdb.py 86.79% <0.00%> (-13.21%) ⬇️
master/buildbot/__init__.py 90.76% <0.00%> (-1.54%) ⬇️
worker/buildbot_worker/__init__.py 91.04% <0.00%> (-1.50%) ⬇️
master/buildbot/db/connector.py 98.83% <0.00%> (-1.17%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 500d6d6...0be5fc8. Read the comment docs.

@rodrigc
Copy link
Contributor Author

rodrigc commented Jan 3, 2021

Trunk fails here:

===============================================================================
[ERROR]
Traceback (most recent call last):
  File "/buildbot/buildbot-job/build/sandbox/lib/python3.8/site-packages/twisted/internet/defer.py", line 1445, in _inlineCallbacks
    result = current_context.run(g.send, result)
  File "/buildbot/buildbot-job/build/master/buildbot/test/unit/scripts/test_start.py", line 117, in test_start_timeout_number_string
    res = yield self.runStart(start_timeout='10')
  File "/buildbot/buildbot-job/build/master/buildbot/test/unit/scripts/test_start.py", line 87, in runStart
    return getProcessOutputAndValue(sys.executable, args=args, env=env)
  File "/buildbot/buildbot-job/build/sandbox/lib/python3.8/site-packages/twisted/internet/utils.py", line 174, in getProcessOutputAndValue
    return _callProtocolWithDeferred(
  File "/buildbot/buildbot-job/build/sandbox/lib/python3.8/site-packages/twisted/internet/utils.py", line 28, in _callProtocolWithDeferred
    reactor.spawnProcess(p, executable, (executable,) + tuple(args), env, path)
  File "/buildbot/buildbot-job/build/sandbox/lib/python3.8/site-packages/twisted/internet/posixbase.py", line 386, in spawnProcess
    args, env = self._checkProcessArgs(args, env)
  File "/buildbot/buildbot-job/build/sandbox/lib/python3.8/site-packages/twisted/internet/base.py", line 1088, in _checkProcessArgs
    _arg = argChecker(arg)
  File "/buildbot/buildbot-job/build/sandbox/lib/python3.8/site-packages/twisted/internet/base.py", line 1074, in argChecker
    arg = arg.encode(defaultEncoding)
builtins.TypeError: encode() argument 'encoding' must be str, not None
buildbot.test.unit.scripts.test_start.TestStart.test_start_timeout_number_string
-------------------------------------------------------------------------------

@rodrigc
Copy link
Contributor Author

rodrigc commented Jan 3, 2021

Testing this branch in Twisted https://github.com/twisted/twisted/tree/10069-rodrigc-encoding,
I could get all the buildbot tests to pass: https://buildbot.buildbot.net/#/builders/87/builds/3

@p12tic
Copy link
Member

p12tic commented Feb 19, 2021

twisted/twisted#1502 is now merged, I think we can close this PR too.

@p12tic p12tic closed this Feb 19, 2021
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.

None yet

3 participants