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

Upgrade operator SDK and Add Folder Sync feature #11

Merged
merged 12 commits into from
Aug 24, 2024

Conversation

robertoeguia
Copy link
Contributor

I apologize for the large pull request.

Changelog:

  • Upgrade operator sdk from 1.5.2 to 1.36.0 as mentioned in TODOs after #8 is merged #9
  • Reorganize files to match operator sdk project layout for easier maintainability
  • Upgrade golang from 1.17 to 1.21
  • Upgrade lastpass-cli to from 1.3.3-4 to use 1.5.0
  • Build lastpass-cli binary as part of multi-stage docker build as newer versions are not provided in package repos
  • Add image pull policy configuration in helm charts
  • Add ability to sync entire lastpass folders Synchronize folders #6

Build lastpass-cli as the more recent stable versions are available as distro packages
Generate CRDs and Roles using controller-gen
Reorganized files to match newer operator-sdk layout.
Created crd type files and manifests for lastpass group types
Added ability to sync entire lastpass folder, not including subfolders
Add missing lastpassgroup crd to chart folder for helm deployment
@niqdev
Copy link
Member

niqdev commented Aug 24, 2024

@robertoeguia thanks for you contributions!

@niqdev niqdev merged commit d1d09b7 into edgelevel:master Aug 24, 2024
@niqdev
Copy link
Member

niqdev commented Aug 24, 2024

@robertoeguia do you mind if I ask what's your use case? is this for personal development or work related? Are you interested in helping maintaining this project from time to time or at least getting involved in PR reviews?

@robertoeguia
Copy link
Contributor Author

robertoeguia commented Aug 24, 2024

@niqdev I don't mind at all. This is for personal use and development. I am interested in maintaining and being involved in this project.

@niqdev
Copy link
Member

niqdev commented Aug 27, 2024

sorry I missed the notification, that's great. as soon as I get a chance I'll setup the action to automatically release the latest docker image when changes are pushed in the main branch

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants