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

feat: Upgrade flux to 2.0.1 #1403

Merged
merged 8 commits into from
Sep 12, 2023
Merged

feat: Upgrade flux to 2.0.1 #1403

merged 8 commits into from
Sep 12, 2023

Conversation

mesosphere-ci
Copy link
Contributor

@mesosphere-ci mesosphere-ci commented Jul 16, 2023 •
edited by takirala
Loading

@mesosphere-ci mesosphere-ci added ready-for-review ok-to-test Signals mergebot that CI checks are ready to be kicked off slack-notify Send notification to Slack labels Jul 16, 2023
@github-actions github-actions bot added size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. services/kommander-flux and removed size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. labels Jul 16, 2023
@d2iq-mergebot
Copy link
Contributor

This repo has @d2iq-mergebot integration. You can perform the following commands by submitting a comment. Submit a comment with content "@d2iq-mergebot help" to view more detailed help text and examples. Be sure the have a look at the mergebot documentation, too.For help using mergebot, please refer to the README file here: https://github.com/mesosphere/mergebot/blob/main/README.md
Enabled Mergebot commands:
@d2iq-mergebot test all
@d2iq-mergebot test
@d2iq-mergebot override-status
@d2iq-mergebot help
@d2iq-mergebot backport

@coveralls
Copy link

coveralls commented Jul 16, 2023 •
edited
Loading

Pull Request Test Coverage Report for Build 6160850041

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 79.532%
Totals Coverage Status
Change from base Build 6160698076: 0.0%
Covered Lines: 136
Relevant Lines: 171
💛 - Coveralls

@gracedo gracedo mentioned this pull request Jul 17, 2023
Closed
@takirala takirala self-assigned this Aug 3, 2023
@github-actions github-actions bot added the size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. label Aug 3, 2023
@takirala takirala removed the slack-notify Send notification to Slack label Aug 3, 2023
@takirala
Copy link
Contributor

takirala commented Aug 9, 2023

/rebase

@cwyl02
Copy link
Contributor

cwyl02 commented Aug 29, 2023

e2e upgrade is also able to pass now with appmanagement running v1beta2 kustomization

@cwyl02 cwyl02 added ok-to-test Signals mergebot that CI checks are ready to be kicked off and removed ok-to-test Signals mergebot that CI checks are ready to be kicked off labels Aug 31, 2023
@cwyl02 cwyl02 force-pushed the flux-update/2.0.1 branch 4 times, most recently from 53c9ad9 to 19c8922 Compare September 7, 2023 21:43
cbuto
cbuto reviewed Sep 8, 2023
View reviewed changes
Copy link
Contributor

@cbuto cbuto left a comment

Choose a reason for hiding this comment

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

looks good to me

.bloodhound.yml Outdated Show resolved Hide resolved
services/kommander-appmanagement/0.7.0/kommander-appmanagement.yaml Outdated Show resolved Hide resolved
services/kommander/0.7.0/kommander.yaml Outdated Show resolved Hide resolved
mesosphere-teamcity and others added 8 commits September 12, 2023 10:24
feat: Upgrade flux to 2.0.1
76cceee
@takirala
chore: bump license manifest for flux images
ce46207 
Signed-off-by: Tarun Gupta Akirala <takirala@users.noreply.github.com>
@takirala
feat: bump to v1beta2 HelmRepository
3ac8bbd 
Signed-off-by: Tarun Gupta Akirala <takirala@users.noreply.github.com>
@takirala
feat: bump to v1 Kustomization
ee09559 
Signed-off-by: Tarun Gupta Akirala <takirala@users.noreply.github.com>
test: hardcode app mgmt tag
cd41718 
TODO: revert this commit
ci: quick fix validate manifests
9945959 
to infestigate real fix in https://d2iq.atlassian.net/browse/D2IQ-99126
chore: modify comment about bloodhound
c192263
revert: "test: hardcode app mgmt tag"
457c1de 
This reverts commit 5364bfb.
@cwyl02
Copy link
Contributor

cwyl02 commented Sep 12, 2023 •
edited
Loading

admin merging since after backing out cd41718 CI cannot pass without manually overriding dev tag and it has been all green when that commit is there

@cwyl02 cwyl02 merged commit 953a00a into main Sep 12, 2023
12 checks passed
@cwyl02 cwyl02 deleted the flux-update/2.0.1 branch September 12, 2023 14:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@takirala takirala takirala left review comments

@gracedo gracedo gracedo approved these changes

@cwyl02 cwyl02 cwyl02 approved these changes

@cbuto cbuto cbuto approved these changes

Assignees

@takirala takirala

Labels
ok-to-test Signals mergebot that CI checks are ready to be kicked off ready-for-review services/centralized-kubecost services/cert-manager services/dkp-insights services/dkp-insights-management services/gatekeeper services/grafana-loki services/istio services/jaeger services/kommander-flux services/kube-prometheus-stack services/kubefed services/nvidia-gpu-operator services/prometheus-adapter services/rook-ceph services/rook-ceph-cluster services/thanos services/velero size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
7 participants
@mesosphere-ci @d2iq-mergebot @coveralls @takirala @cwyl02 @gracedo @cbuto