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.

Sign up for GitHub

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

Jump to bottom

Release Ready #2903

Merged
merged 104 commits into from
Aug 27, 2024
Merged

Release Ready #2903

merged 104 commits into from
Aug 27, 2024

Conversation

sgoggins
Copy link
Member

Description
Many new things

sgoggins and others added 30 commits August 11, 2024 09:48
@sgoggins
Merge pull request #2885 from chaoss/docker-dev
f0c73cd 
Docker dev: Docker WOrking
@ABrain7710
Set per page to 100
f22613c 
Signed-off-by: Andrew Brain <[email protected]>
@ABrain7710
start splitting out event collections into two classes
8f07cf4
@ABrain7710
define thorough event collection
1fd8266
@ABrain7710
remove old implementation and clean up some things
c15dd09
@ABrain7710
pass params
a3169b0
@ABrain7710
call correct method name
d7c093a
@ABrain7710
other syntax fixes
36ad632
@ABrain7710
only log not mappable events when there are some
347a2e2
@ABrain7710
only add non null contributors
b45b37f
@sgoggins
Merge branch 'dev' into dev-fixes
7f41d24 
Signed-off-by: Sean P. Goggins <[email protected]>
@ABrain7710
fix: add url not found catching
2b5ce1a
@sgoggins
Merge pull request #2897 from chaoss/dev-fixes
9d65e24 
Dev fixes for event collection
@Ulincsys
Pull relevant mods from admin-changes
36ae56c 
Signed-off-by: Ulincsys <[email protected]>
@Ulincsys
initial work on jumpstart project
f0e27a3 
Signed-off-by: Ulincsys <[email protected]>
@Ulincsys
Implement decoding and blocking
709e771 
Signed-off-by: Ulincsys <[email protected]>
@Ulincsys
add TUI, expand server basis
d7c6a87 
Signed-off-by: Ulincsys <[email protected]>
@Ulincsys
update encoding
3bac827 
Signed-off-by: Ulincsys <[email protected]>
@Ulincsys
continue work on concurrency
cf6c9ec 
Signed-off-by: Ulincsys <[email protected]>
@Ulincsys
mvp
121ee03 
Signed-off-by: Ulincsys <[email protected]>
@Ulincsys
Fix improper client shutdown on some platforms
761cc50 
Signed-off-by: Ulincsys <[email protected]>
@Ulincsys
Add DB check before parsing start command
e920890 
Signed-off-by: Ulincsys <[email protected]>
@ABrain7710 @Ulincsys
Add github graphql data access'
2067d44 
Signed-off-by: Andrew Brain <[email protected]>
Signed-off-by: Ulincsys <[email protected]>
@ABrain7710 @Ulincsys
Improve paginate_resource logic
140ea06 
Signed-off-by: Andrew Brain <[email protected]>
Signed-off-by: Ulincsys <[email protected]>
@ABrain7710 @Ulincsys
Remove unused method
5577ce0 
Signed-off-by: Andrew Brain <[email protected]>
Signed-off-by: Ulincsys <[email protected]>
@ABrain7710 @Ulincsys
Move repo_info to new graphql endpoint
06d2c06 
Signed-off-by: Andrew Brain <[email protected]>
Signed-off-by: Ulincsys <[email protected]>
@ABrain7710 @Ulincsys
Migrate pr files to github_graphql_data_access
fde0ec4 
Signed-off-by: Andrew Brain <[email protected]>
Signed-off-by: Ulincsys <[email protected]>
@ABrain7710 @Ulincsys
Indent client'
06bc536 
Signed-off-by: Andrew Brain <[email protected]>
Signed-off-by: Ulincsys <[email protected]>
@ABrain7710 @Ulincsys
Remove strip
2e05618 
Signed-off-by: Andrew Brain <[email protected]>
Signed-off-by: Ulincsys <[email protected]>
@ABrain7710 @Ulincsys
Define keys as list
ec71977 
Signed-off-by: Andrew Brain <[email protected]>
Signed-off-by: Ulincsys <[email protected]>
sgoggins and others added 29 commits August 27, 2024 18:06
@sgoggins @Ulincsys
bug fix for events
e4a0a3e 
Signed-off-by: Sean P. Goggins <[email protected]>
Signed-off-by: Ulincsys <[email protected]>
@ABrain7710 @Ulincsys
Handle url not found in contributor breadth worker
cf99b23 
Signed-off-by: Andrew Brain <[email protected]>
Signed-off-by: Ulincsys <[email protected]>
@sgoggins @Ulincsys
docker fun
44b1c62 
Signed-off-by: Sean P. Goggins <[email protected]>
Signed-off-by: Ulincsys <[email protected]>
@sgoggins @Ulincsys
failed docker config
456b0f5 
Signed-off-by: Sean P. Goggins <[email protected]>
Signed-off-by: Ulincsys <[email protected]>
@sgoggins @Ulincsys
saving failed config
07c2c38 
Signed-off-by: Sean P. Goggins <[email protected]>
Signed-off-by: Ulincsys <[email protected]>
@sgoggins @Ulincsys
Docker compose script now workinggit add docker/
f0cad14 
Signed-off-by: Sean P. Goggins <[email protected]>
Signed-off-by: Ulincsys <[email protected]>
@sgoggins @Ulincsys
removed old file
6a6ba73 
Signed-off-by: Sean P. Goggins <[email protected]>
Signed-off-by: Ulincsys <[email protected]>
@sgoggins @Ulincsys
debugging a couple errors. One with a missing import, the other tryin…
56808aa 
…g to understand event API limits

Signed-off-by: Sean P. Goggins <[email protected]>
Signed-off-by: Ulincsys <[email protected]>
@sgoggins @Ulincsys
possible fix for commit messages
ca0d4b2 
Signed-off-by: Sean P. Goggins <[email protected]>
Signed-off-by: Ulincsys <[email protected]>
@sgoggins @Ulincsys
changed page_count > 300 to a warning instead of an exception
f512443 
Signed-off-by: Sean P. Goggins <[email protected]>
Signed-off-by: Ulincsys <[email protected]>
@sgoggins @Ulincsys
updated unterminated string error
457b03d 
Signed-off-by: Sean P. Goggins <[email protected]>
Signed-off-by: Ulincsys <[email protected]>
@sgoggins @Ulincsys
Fixing NPM
d7d435b 
Signed-off-by: Sean P. Goggins <[email protected]>
Signed-off-by: Ulincsys <[email protected]>
@sgoggins @Ulincsys
events page count seems to top out at 1,000, not 300
21660e1 
Signed-off-by: Sean P. Goggins <[email protected]>
Signed-off-by: Ulincsys <[email protected]>
@sgoggins @Ulincsys
update to events to use correct column name
e665aec 
Signed-off-by: Sean P. Goggins <[email protected]>
Signed-off-by: Ulincsys <[email protected]>
@sgoggins @Ulincsys
update to augur event collection
b80a480 
Signed-off-by: Sean P. Goggins <[email protected]>
Signed-off-by: Ulincsys <[email protected]>
@sgoggins @Ulincsys
logging event so we know what's supposed to be there
c35f449 
Signed-off-by: Sean P. Goggins <[email protected]>
Signed-off-by: Ulincsys <[email protected]>
@sgoggins @Ulincsys
Looking at event logs, it appears that issue is not always contained …
a9830ea 
…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]>
Signed-off-by: Ulincsys <[email protected]>
@sgoggins @Ulincsys
fixing pickling error caused by not consistently importing our Metada…
de320b2 
…taException class (I think)

Signed-off-by: Sean P. Goggins <[email protected]>
Signed-off-by: Ulincsys <[email protected]>
@sgoggins @Ulincsys
additional update for when issue is not an object in an event respons…
dbdb7e3 
…e to the API

Signed-off-by: Sean P. Goggins <[email protected]>
Signed-off-by: Ulincsys <[email protected]>
@sgoggins @Ulincsys
Trying to fix issue when the committer count API returns no value on …
4c45d8f 
…empty repos in repo_info

Signed-off-by: Sean P. Goggins <[email protected]>
Signed-off-by: Ulincsys <[email protected]>
@sgoggins @Ulincsys
exception handling
ed50be2 
Signed-off-by: Sean P. Goggins <[email protected]>
Signed-off-by: Ulincsys <[email protected]>
@sgoggins @Ulincsys
trying to fix the empty committer count issue
f510da4 
Signed-off-by: Sean P. Goggins <[email protected]>
Signed-off-by: Ulincsys <[email protected]>
@sgoggins @Ulincsys
added some comments explaining the query_committers_count method changes
82b9159 
Signed-off-by: Sean P. Goggins <[email protected]>
Signed-off-by: Ulincsys <[email protected]>
@sgoggins @Ulincsys
Making repo GONE log message more clear.
8967212 
Signed-off-by: Sean P. Goggins <[email protected]>
Signed-off-by: Ulincsys <[email protected]>
@sgoggins @Ulincsys
Working to get more information on this error:
5705644 
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]>
Signed-off-by: Ulincsys <[email protected]>
@sgoggins @Ulincsys
formatting exception for OpenSSF Scorecard pickling error more typically
97fee7e 
Signed-off-by: Sean P. Goggins <[email protected]>
Signed-off-by: Ulincsys <[email protected]>
@Ulincsys
fix for missing comma, fix misspelling
ebca962 
Signed-off-by: Ulincsys <[email protected]>
@sgoggins
Merge branch 'dev' into jumpstart
491b688 
Signed-off-by: Sean P. Goggins <[email protected]>
@sgoggins
Merge pull request #2902 from chaoss/jumpstart
aae2f19 
Jumpstart
@sgoggins sgoggins merged commit 6ffbcc4 into augur-release-0.76.2 Aug 27, 2024
4 of 5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@sgoggins @ABrain7710 @Ulincsys @IsaacMilarky