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

Add Flux OCI support #358

Closed
wants to merge 6 commits into from
Closed

Add Flux OCI support #358

wants to merge 6 commits into from

Conversation

kingdonb
Copy link
Collaborator

@kingdonb kingdonb commented Aug 2, 2022

Fix: #309

This is released in an edge build, so far it seems like there are no issues with backward or forward compatibility.

Component PRs:

Testers needed! Please review kingdonb/podinfo, we have these GitHub Actions workflows for example:

If your architecture or OS is different than CI you may also need to clone fluxcd/flux2, checkout from the oci branch, and run make to build the CLI for your workstation. (Otherwise, you can retrieve a CLI binary that is built from CI, it is published only once as a build artifact on the first "Build Flux OCI Binary (Pre-release)" build job that runs. It will be rebuilt again if the prerelease oci branch is changed.)

It is expected for this workflow to vary slightly until we can solidify the guidance around how to use the new OCI feature more, and provide more concrete examples or even a "Flux cookbook."

See also (the fluxcd/flux2 branch oci) where this workflow derives the prerelease Flux binary from:

This is WIP (but basically finished now) and can merge after the next Flux minor release series has landed, v0.32.x.

Kingdon Barrett added 3 commits August 2, 2022 18:13
Signed-off-by: Kingdon Barrett <kingdon@weave.works>
This basically works 🎉

- [X] OCI Source nodes in source tree view
- [X] OCI Source hover text shows minimum information
- [ ] OCI Source hover text shows all relevant fields
- [X] Right click context entries work for OCI sources
- [ ] Create Source workflow includes OCIRepository kind
- [ ] Kustomizations with OCIRepository sources are tested

Signed-off-by: Kingdon Barrett <kingdon@weave.works>
One of these will be set at least, or the OCIRepository should default
to pulling the 'latest' tag (which is fine if you like, sure, but that
would not be GitOps anymore...)

Signed-off-by: Kingdon Barrett <kingdon@weave.works>
@kingdonb kingdonb added this to the 0.21 milestone Aug 2, 2022
@kingdonb kingdonb marked this pull request as draft August 3, 2022 02:29
@kingdonb
Copy link
Collaborator Author

kingdonb commented Aug 3, 2022

The outstanding work items that remain from #355 and off the top of my head to complete here are:

  • Create Source workflow includes OCIRepository kind We are not going to do this for now, let's re-think the Create Source workflow
  • OCIRepository sources are tested 👍 (08b1581)
  • Kustomizations with OCIRepository sources are tested together - e2e I did not implement this, we should add more basic integration or unit tests instead. I did not want to make the release workflow longer as it already takes about 15 minutes depending on how long the marketplace takes to update.
  • Add documentation to the getting started page and update screenshots - still WIP, add to 0.21.x milestone
  • Add more documentation, and probably a full narrated screencast in the docs - still WIP, add to 0.21.x milestone

Kingdon Barrett added 3 commits August 2, 2022 22:59
The flux cli from "Build Flux OCI Binary (Pre-release)" can be used to
test against a Flux with the new OCI support.

Signed-off-by: Kingdon Barrett <kingdon@weave.works>
(it should be possible for us to push a prerelease that is tested
against the new oci release and exercises the features)

Signed-off-by: Kingdon Barrett <kingdon@weave.works>
Signed-off-by: Kingdon Barrett <kingdon@weave.works>
@kingdonb
Copy link
Collaborator Author

This one can close, it was superseded by

@kingdonb kingdonb closed this Aug 11, 2022
@kingdonb kingdonb deleted the add-flux-oci branch August 21, 2023 15:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Feature: Support for Flux OCI features
1 participant