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

BB Extension Flux deprecation on 2.7.0+ #1932

Closed
mjnagel opened this issue Jul 31, 2023 · 0 comments · Fixed by #1933
Closed

BB Extension Flux deprecation on 2.7.0+ #1932

mjnagel opened this issue Jul 31, 2023 · 0 comments · Fixed by #1933
Labels
possible-bug 🐛

Comments

@mjnagel
Copy link
Contributor

mjnagel commented Jul 31, 2023

Environment

Device and OS: ec2 instance, Ubuntu
App version: zarf 0.28.3, BB 2.7.0
Kubernetes distro being used: k3d

Steps to reproduce

Deploy BB 2.7.0 via the extension and examine the debug log. You will see a warning about the deprecation:

└ (/home/runner/work/zarf/zarf/src/pkg/message/message.go:137)
  ⠦  Attempting chart installation (3s)  DEBUG   2023-07-31T16:15:56Z  -  "level"=0 "msg"="v1beta2 GitRepository is deprecated, upgrade to v1\n"                                                                                                                                 
└ (/home/runner/work/zarf/zarf/src/pkg/message/message.go:137)
  ⠦  creating 13 resource(s)  DEBUG   2023-07-31T16:15:56Z  -  "level"=0 "msg"="v1beta2 GitRepository is deprecated, upgrade to v1\n"                                                                                                                                            
└ (/home/runner/work/zarf/zarf/src/pkg/message/message.go:137)

Expected result

No deprecated API Versions should be used.

Actual Result

Warning about gitrepo being on deprecated version
@mjnagel mjnagel mentioned this issue Jul 31, 2023
Merged
5 tasks
Racer159 added a commit that referenced this issue Aug 8, 2023
@mjnagel @Racer159
@Noxsios @cmwylie19
Big Bang extension flux api version update (#1933)
ca6546b 
## Description

Updates the Flux GitRepo version to v1.

Not sure if this is the right/desired fix or if this should be more
backwards compatible (similar to the way credentials are handled for BB
1/2.

## Related Issue

Fixes #1932

## Type of change

- [x] Bug fix (non-breaking change which fixes an issue)
- [ ] New feature (non-breaking change which adds functionality)
- [ ] Other (security config, docs update, etc)

## Checklist before merging

- [x] Test, docs, adr added or updated as needed
- [x] [Contributor Guide
Steps](https://github.com/defenseunicorns/zarf/blob/main/CONTRIBUTING.md#developer-workflow)
followed

---------

Co-authored-by: Wayne Starr <Racer159@users.noreply.github.com>
Co-authored-by: razzle <harry@razzle.cloud>
Co-authored-by: Case Wylie <cmwylie19@defenseunicorns.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
possible-bug 🐛
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Big Bang extension flux api version update zarf-dev/zarf
1 participant
@mjnagel