lix-project/buildbot-nix
14
0
Fork 1
Code Issues 13 Pull requests 6 Projects Releases Packages Wiki Activity

chore(*): refactor the whole code #8

Merged
raito merged 24 commits from refactor into main 2024-10-05 18:31:28 +00:00
Conversation 3 Commits 24 Files changed 8 +314 -535
raito commented 2024-05-06 16:29:44 +00:00
Owner
Copy link

It is long due to move past the nice prototype, I will be trying to refactor the code, split it.

On my todolist, I have:

  • cleanup the GerritConfigurator options in the nix module and expose them
  • static configuration of projects on Gerrit
    - [ ] autodiscovery of projects on Gerrit dynamically moved to a next PR
  • make it clear what is the entrypoint accepted: .hydraJobs from what I understand
    - [ ] split up the super large __init__.py into multiple logical components moved to a next PR

Tested on buildbot.forkos.org.

It is long due to move past the nice prototype, I will be trying to refactor the code, split it. On my todolist, I have: - [x] cleanup the GerritConfigurator options in the nix module and expose them - [x] static configuration of projects on Gerrit ~~- [ ] autodiscovery of projects on Gerrit dynamically~~ moved to a next PR - [x] make it clear what is the entrypoint accepted: `.hydraJobs` from what I understand ~~- [ ] split up the super large `__init__.py` into multiple logical components~~ moved to a next PR Tested on buildbot.forkos.org.
Signed-off-by: Raito Bezarius <raito@lix.systems>
Signed-off-by: Raito Bezarius <raito@lix.systems>
Signed-off-by: Raito Bezarius <raito@lix.systems>
Instead of just commenting them out.

Signed-off-by: Raito Bezarius <raito@lix.systems>
raito changed title from chore(*): refactor the whole code to WIP: chore(*): refactor the whole code 2024-05-06 16:40:08 +00:00 
Signed-off-by: Raito Bezarius <raito@lix.systems>
It was relying on GitHub stuff which we don't have and is not an option
we want to support.

If we wanted to do it, we would rather use S3 directly.

Signed-off-by: Raito Bezarius <raito@lix.systems>
Previously, we needed to hardcode the URL for private SSH keys,
this is cleaned up and we can iterate over each project for its
configuration.

Configuration is at deployment time.

Signed-off-by: Raito Bezarius <raito@lix.systems>
Signed-off-by: Raito Bezarius <raito@lix.systems>
This makes it easier to make it configurable, this is step 1.

Signed-off-by: Raito Bezarius <raito@lix.systems>
Just iterate over all project names.

Signed-off-by: Raito Bezarius <raito@lix.systems>
Signed-off-by: Raito Bezarius <raito@lix.systems>
Signed-off-by: Raito Bezarius <raito@lix.systems>
Signed-off-by: Raito Bezarius <raito@lix.systems>
Signed-off-by: Raito Bezarius <raito@lix.systems>
Signed-off-by: Raito Bezarius <raito@lix.systems>
Worked around in our original deployment, here's a nicer way to set it.

Signed-off-by: Raito Bezarius <masterancpp@gmail.com>
So that it's possible to plug another OAuth2 instance.

Signed-off-by: Raito Bezarius <masterancpp@gmail.com>
Signed-off-by: Raito Bezarius <masterancpp@gmail.com>
Signed-off-by: Raito Bezarius <masterancpp@gmail.com>
Signed-off-by: Raito Bezarius <masterancpp@gmail.com>
Signed-off-by: Raito Bezarius <masterancpp@gmail.com>
Signed-off-by: Raito Bezarius <masterancpp@gmail.com>
Signed-off-by: Raito Bezarius <masterancpp@gmail.com>
Signed-off-by: Raito Bezarius <masterancpp@gmail.com>
Signed-off-by: Raito Bezarius <masterancpp@gmail.com>
raito changed title from WIP: chore(*): refactor the whole code to chore(*): refactor the whole code 2024-07-19 17:24:53 +00:00
raito commented 2024-07-19 17:25:09 +00:00
Author
Owner
Copy link

This is good enough for a merge, IMHO, @puck, can I ask you for a review?

This is good enough for a merge, IMHO, @puck, can I ask you for a review?
jade commented 2024-10-04 21:09:03 +00:00
Owner
Copy link

I have rebased this. It can be merged with your signoff @raito.

I have rebased this. It can be merged with your signoff @raito.
raito merged commit 48828cb33f into main 2024-10-05 18:31:28 +00:00
raito deleted branch refactor 2024-10-05 18:31:29 +00:00
raito commented 2024-10-05 18:31:42 +00:00
Author
Owner
Copy link

It's now time to get the non-flakes PR as well.

It's now time to get the non-flakes PR as well.
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
lix-project/buildbot-nix!8
No description provided.