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

Initial version of harbor day2 operator #3

Merged
merged 49 commits into from
Apr 17, 2024
Merged

Initial version of harbor day2 operator #3

merged 49 commits into from
Apr 17, 2024

Conversation

peters-david
Copy link
Contributor

Initial working version of the Harbor Day2 Operator.

It can synchronize harbor projects, registries, members, webhooks and robot accounts.
The python code is built using nuitka.

alfsch and others added 30 commits September 20, 2023 14:50
@alfsch
first version of harbor configurator built with nuitka
5a273d8 
- fixed some linting issues
- added basic implementation out of prototyping project
@alfsch
fix image test
f6fb4fe
@alfsch
get one image pushed
0c2039a
@tobpil
set config folder path with env variable
04dfe02
@tobpil
implement registry syncing
c5f146b
@tobpil
implement project syncing
fe4f4ce
@tobpil
implement robot account syncing
1717244
@tobpil
implement webhook syncing
0120e84
@tobpil
improve console output readability
8c68075
@tobpil
add robot_name_prefix as env variable
aafa118
@tobpil
split long string lines
026dbdf
@tobpil
define environment on top
ff2db19
@tobpil
add instruction for when robot account is project defined
4d35a5c
@tobpil
fix console output
5dc0eaa
@tobpil
do not delete non empty projects
adc2595
@tobpil
check if project is empty before deleting
bc29b04
@tobpil
add argument parser and use old password env variable
f40b84e
@tobpil
use robot secrets from env variables
d22f15e
@tobpil
call help page to test native compilation in build pipeline
a6f6263
@tobpil
fix linting issues
61460f8
@tobpil
define global client in main function
6438ada
@tobpil
fix linting warning
d5cf226
@tobpil
check if native compilation works before pushing
0c2f5db
@tobpil
fix target_robot attribute name
24b50c5
@steadyk
Update dependencies
01080ff
@peters-david
added minimal error handling for bad request on robot sync
a3f1a24
@peters-david
added harbor error print to conflict error
21c80dc
@peters-david
print harbor error while respecting linter rules
24ad297
@steadyk
Fix robot account names which contains namespace name
41f6b86
@steadyk
Add paranthesis to avoid wrong namespace allocation
53756e5
steadyk and others added 19 commits March 6, 2024 17:20
@steadyk
Add missing robot_name; simplify usage
f07c4fa
@steadyk
Refactor naming
860e1cb
@steadyk
Fix quoting issues and add debug log
fcbd2c5
@steadyk
Adding needed awaits; rewert unneeded quoting removal
34662e9
@steadyk
Remove unneeded line, add debug log
357cdc8
@peters-david
changed identity operator to equality operator and changed order of t…
ebe413e 
…arget robot usage
@peters-david
changed dict attribute access
a4baf16
@peters-david
fix missing closing bracket
6281694
@peters-david
changed attribute acces in construct name method
1b95e5b
@peters-david
changed single quotes to double quotes to have clear f string
9e42aac
@peters-david
printing current robot names to test if indices come from harbor
ce8eef7
@peters-david
expand robot level to system and projects using query
4d72b40
@peters-david
separate queries for project and system
1eb71e7
@peters-david
added robot queries to include all robots from all projects
ea1708f
@peters-david
fix typo in variable name
248d7bf
@peters-david
bigger pagination page size
1966560
@peters-david
reduce pagination page size to 100 for harbor
7c595bc
@peters-david
remove unnecessary robot sync prints
563f3b4
@peters-david
remove page_size and add explicit limit=None to fetch all resources
3952719
@peters-david peters-david requested a review from steadyk April 11, 2024 09:21
steadyk
steadyk approved these changes Apr 11, 2024
View reviewed changes
Copy link
Contributor

@steadyk steadyk left a comment

Choose a reason for hiding this comment

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

Should be fine, since we tested it locally and on dev!

@peters-david peters-david merged commit 12cc25b into main Apr 17, 2024
2 checks passed
@peters-david peters-david deleted the initial-version branch April 17, 2024 07:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@steadyk steadyk steadyk approved these changes

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
@peters-david @steadyk @alfsch @tobpil