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

Docker dev fix: Fixes to commit comments and events #2891

Merged
merged 38 commits into from
Aug 15, 2024
Merged

Conversation

sgoggins
Copy link
Member

Description

  • Fixes commit comments not decoding hex
  • Fixes events where limit set to check at 300, but API returning 1,000 pages
  • Fixes a typo in the events SQL: gh_issue_number

Signed commits

  • Yes, I signed my commits.

sgoggins and others added 26 commits August 5, 2024 09:20
Signed-off-by: Sean P. Goggins <[email protected]>
Signed-off-by: Sean P. Goggins <[email protected]>
Signed-off-by: Sean P. Goggins <[email protected]>
Signed-off-by: Sean P. Goggins <[email protected]>
Signed-off-by: Sean P. Goggins <[email protected]>
Signed-off-by: Sean P. Goggins <[email protected]>
Signed-off-by: Sean P. Goggins <[email protected]>
Signed-off-by: Sean P. Goggins <[email protected]>
Signed-off-by: Sean P. Goggins <[email protected]>
Signed-off-by: Sean P. Goggins <[email protected]>
Dev update to docker fixes
Handle url not found in contributor breadth worker
Signed-off-by: Sean P. Goggins <[email protected]>
Signed-off-by: Sean P. Goggins <[email protected]>
Signed-off-by: Sean P. Goggins <[email protected]>
Signed-off-by: Sean P. Goggins <[email protected]>
…g to understand event API limits

Signed-off-by: Sean P. Goggins <[email protected]>
Signed-off-by: Sean P. Goggins <[email protected]>
Signed-off-by: Sean P. Goggins <[email protected]>
Signed-off-by: Sean P. Goggins <[email protected]>
@sgoggins sgoggins added the bug-fix Fixes a bug label Aug 12, 2024
@sgoggins sgoggins self-assigned this Aug 12, 2024
import logging
import traceback
import logging
import traceback

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[pylint] reported by reviewdog 🐶
W0611: Unused import traceback (unused-import)

Signed-off-by: Sean P. Goggins <[email protected]>
…in the data returned from the events API. It DOES look like what is returned may be a partially complte set in some cases.

Signed-off-by: Sean P. Goggins <[email protected]>
…taException class (I think)

Signed-off-by: Sean P. Goggins <[email protected]>
…empty repos in repo_info

Signed-off-by: Sean P. Goggins <[email protected]>
Signed-off-by: Sean P. Goggins <[email protected]>
Traceback (most recent call last):
  File '/home/ubuntu/github/virtualenvs/hosted/lib/python3.11/site-packages/billiard/pool.py', line 366, in workloop
    put((READY, (job, i, result, inqW_fd)))
  File '/home/ubuntu/github/virtualenvs/hosted/lib/python3.11/site-packages/billiard/queues.py', line 366, in put
    self.send_payload(ForkingPickler.dumps(obj))
                      ^^^^^^^^^^^^^^^^^^^^^^^^^
  File '/home/ubuntu/github/virtualenvs/hosted/lib/python3.11/site-packages/billiard/reduction.py', line 56, in dumps
    cls(buf, protocol).dump(obj)
billiard.pool.MaybeEncodingError: Error sending result: ''(1, <ExceptionInfo: MetadataException('\'checks\' | Additional metadata: required_output: {}')>, None)''. Reason: ''PicklingError('Can\'t pickle <class \'augur.tasks.util.metadata_exception.MetadataException\'>: it\'s not the same object as augur.tasks.util.metadata_exception.MetadataException')''.

Signed-off-by: Sean P. Goggins <[email protected]>
tracer = ''.join(traceback.format_exception(type(e), e, e.__traceback__))
logger.warning(f'Full stack trace of OpenSSF scorecard error: {tracer}')
raise MetadataException(f"An error occurred while generating the scorecard: {str(e)}")

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[pylint] reported by reviewdog 🐶
E1120: No value for argument 'additional_metadata' in constructor call (no-value-for-parameter)

@sgoggins sgoggins merged commit b778fcd into dev Aug 15, 2024
8 of 9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug-fix Fixes a bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants