pyeventsourcing / eventsourcing

A library for event sourcing in Python.
https://eventsourcing.readthedocs.io/
BSD 3-Clause "New" or "Revised" License
1.41k stars 129 forks source link

postgresql expects integer for lock_timeout #236

Closed bernardotorres closed 2 years ago

bernardotorres commented 2 years ago

Tested this live and with this patch, it works.

bernardotorres commented 2 years ago

Tests passed, but getting some errors related to Coveralls

Trying submitting coverage with service_name: github...
cd .
Failed submitting coverage with service_name: github
Traceback (most recent call last):
  File "/usr/local/lib/python3.9/site-packages/coveralls/api.py", line 259, in submit_report
    response.raise_for_status()
  File "/usr/local/lib/python3.9/site-packages/requests/models.py", line 943, in raise_for_status
    raise HTTPError(http_error_msg, response=self)
requests.exceptions.HTTPError: 405 Client Error: Not Allowed for url: https://coveralls.io/api/v1/jobs

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/src/entrypoint.py", line 82, in run_coveralls
    result = coveralls.wear()
  File "/usr/local/lib/python3.9/site-packages/coveralls/api.py", line 226, in wear
    return self.submit_report(json_string)
  File "/usr/local/lib/python3.9/site-packages/coveralls/api.py", line 262, in submit_report
    raise CoverallsException(
coveralls.exception.CoverallsException: Could not submit coverage: 405 Client Error: Not Allowed for url: https://coveralls.io/api/v1/jobs
cd /github/workspace
Trying submitting coverage with service_name: github-actions...
cd .
Failed submitting coverage with service_name: github-actions
Traceback (most recent call last):
  File "/usr/local/lib/python3.9/site-packages/coveralls/api.py", line 259, in submit_report
    response.raise_for_status()
  File "/usr/local/lib/python3.9/site-packages/requests/models.py", line 943, in raise_for_status
    raise HTTPError(http_error_msg, response=self)
requests.exceptions.HTTPError: 405 Client Error: Not Allowed for url: https://coveralls.io/api/v1/jobs

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/src/entrypoint.py", line 82, in run_coveralls
    result = coveralls.wear()
  File "/usr/local/lib/python3.9/site-packages/coveralls/api.py", line 226, in wear
    return self.submit_report(json_string)
  File "/usr/local/lib/python3.9/site-packages/coveralls/api.py", line 262, in submit_report
    raise CoverallsException(
coveralls.exception.CoverallsException: Could not submit coverage: 405 Client Error: Not Allowed for url: https://coveralls.io/api/v1/jobs
cd /github/workspace
Failed to submit coverage
ExitCode.FAILURE
johnbywater commented 2 years ago

I was going to ask you to check this one too, but didn't want to trouble you. So thank you very much for taking the initiative!

Yes, coveralls's API has been a bit less than completely reliable in recent times. Re-running all the jobs normally helps...

bernardotorres commented 2 years ago

Maybe I didn't look properly, but it seems i don't have permission to re-run the tests.

johnbywater commented 2 years ago

I reran them twice and got the same error twice, which is unusual. I think we are using the latest version of their tool so I don't know what else to do. Somebody keeps breaking their API and I don't understand it. It used to always just work. Maybe it will work tomorrow?

johnbywater commented 2 years ago

If you click through to the "details" there is a button top right. Screenshot_20210918_235600

johnbywater commented 2 years ago

Still not working today 😬

bernardotorres commented 2 years ago

That's probably only visible to repo maintainers. image

Unknown22 commented 2 years ago

The same problem is for idle_in_transaction_session_timeout in the same class

coveralls commented 2 years ago

Pull Request Test Coverage Report for Build 1248053682


Totals Coverage Status
Change from base Build 1243278855: 0.0%
Covered Lines: 2277
Relevant Lines: 2277

💛 - Coveralls
johnbywater commented 2 years ago

Looks like coveralls fixed their stuff 👍