Skip to content

Bundle staging PRs #213

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
Closed
wants to merge 218 commits into from
Closed

Bundle staging PRs #213

wants to merge 218 commits into from

Conversation

@trz42
Copy link
Contributor

@trz42 trz42 commented Apr 22, 2025

Attempt to bundle tarballs belonging to the same build PR into a single staging PR.

Also adds code linting and a CI check for it.

Draft because

  • it's just a first, likely not final or complete, version
  • needs to be tested

@trz42 trz42 added the enhancement New feature or request label Apr 22, 2025
truib added 26 commits April 22, 2025 10:45
@truib
requirements.txt is not needed
dd9b71a
@truib
fix flake8 issues in automated_ingestion.py
2cfde96
@truib
fix flake8 issues in check-stratum-servers.py
0cb2622
@truib
incremental updates through testing
7214d27
@truib
add missing functions
946dd65
@truib
mark tarballs in group as new initially
51894ec
@truib
always use group branch only
ea51a5e
@truib
add a bit debug output
7db3dbe
@truib
improve logging
9ef3535
@truib
more improvements to logging
d860d1e
@truib
tweak logging
18cf44d
@truib
more logging output for downloads and signature verification
4a6fecc
@truib
add capability to run verify script in a container
e1b1ee6
@truib
improve branch naming and only add files to staged dir in main branch
56a9d68
@truib
don't stop after staging files
935025f
@truib
add more log info when moving metadata files
682ddbd
@truib
need to handle first tarball differently
412cc46
@truib
move from staged to approval plus a little more logging
d0f0d26
@truib
fix missing function
c2c456d
@truib
improve get_metadata_path
5cad5ed
@truib
enhance logging capabilities
d045aa5
@truib
add more logging scopes, convert logging calls and improve code reada…
f9fd34e 
…bility
@truib
show function entry/exit at info level
aab9f6a
@truib
tweak func leave msg and add context info
678f0d7
@truib
fix shown file components and illustrate call stack depth
7ecb9be
@truib
convert logging calls in automated_ingestion.py
c65d1f7
truib added 26 commits June 21, 2025 17:17
@truib
improve logging when processing ingestion failure
98c0409
@truib
add traceback when catching exception
da9cc94
@truib
convert Path to str
54062a3
@truib
bump sequence number to 11
bb6021e
@truib
1st step to make sequence numbers non-hardcoded
de00cee
@truib
rename target_dir to pull_request_dir
b8240dd
@truib
remove function to create symlink
345bbb7
@truib
remove function to obtain branch name from sequence number
e800415
@truib
remove one use of _get_fixed_sequence_number
d717527
@truib
improve ways to obtain pull request directory
868b23b
@truib
clarify how to determine pull_request_dir
dd0e8bb
@truib
remove need for determining sequence number in _create_task_summary
20a7ae8
@truib
add function to determine sequence number and use the function
814cd50
@truib
improve logging when determining task state
cff761e
@truib
add logging and consider 'target_dir' attr name
d85e017
@truib
add logging for determining sequence number and pull request
cf08115
@truib
print head refs for all PRs
f94f69d
@truib
fix PR head ref logging
06faa4b
@truib
do not use login when searching for PRs
9979adf
@truib
improve function to determine PR
af021ae
@truib
fix branch name
251b60e
@truib
restructure logging when determining PRR”
2275007
@truib
change way to determine PRs for branch name
cffc319
@truib
little code cleanup
29d17fc
@truib
reformat PR body and make format configurable
be9443c
@truib
make config dictionary available for analyse_contents
3713639
This was referenced Jun 27, 2025
Open
Merged
@trz42
Copy link
Contributor Author

trz42 commented Sep 4, 2025

Closing as #232 has been merged.

@trz42 trz42 closed this Sep 4, 2025
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

enhancement New feature or request

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@trz42 @truib