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

Rebase master with 0.10.0 and 0.9.1 releases #1403

Merged

Conversation

reginapizza
Copy link
Collaborator

What type of PR is this?

Uncomment only one /kind line, and delete the rest.
For example, > /kind bug would simply become: /kind bug

/kind bug
/kind chore
/kind cleanup
/kind failing-test
/kind enhancement
/kind documentation
/kind code-refactoring

What does this PR do / why we need it:

Have you updated the necessary documentation?

  • Documentation update is required by this PR.
  • Documentation has been updated.

Which issue(s) this PR fixes:

Fixes #?

How to test changes / Special notes to the reviewer:

Copy link
Member

@jgwest jgwest left a comment

Choose a reason for hiding this comment

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

Hey, what is this PR for? The list of commits looks scary 😅

@reginapizza
Copy link
Collaborator Author

reginapizza commented Jun 5, 2024

@jgwest yes, I agree :') I was rebasing the master branch with the changes in the release-0.10 branch. After I released 0.10.0 I didn't update the master branch right away (slipped my mind with vacation and the amount of time it took to get the olm catalog PR's merged- my bad) but it seems like there were a lotttt of other changes also not synced to master branch from release-0.10 branch? I've already taken a look at the 0.10.0 changes that I made and I think there's okay, but I've requested Anand and Abhishek's reviews to make sure everything else looks right before we consider merging it. If you also wouldn't mind taking a look, the more eyes on it the better!

@jgwest
Copy link
Member

jgwest commented Jun 5, 2024

@reginapizza Strangely, the 'Files Changed' tab of this PR seems to be reporting changes to master branch that are already on master branch. 🤔

@reginapizza reginapizza force-pushed the rebase-master-0.10.0 branch from 48bca8c to ebc1aef Compare June 5, 2024 12:31
@reginapizza
Copy link
Collaborator Author

Okay I re-synced from master again (I can see I pulled from it originally, so I'm not really sure why it did all those commits over again) and now I think it looks much better! Thanks for the tip @jgwest

@jgwest jgwest self-requested a review June 5, 2024 12:52
Copy link
Collaborator

@anandf anandf left a comment

Choose a reason for hiding this comment

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

LGTM

@reginapizza reginapizza changed the title Rebase master with release-0.10.0 branch Rebase master with 0.10.0 and 0.9.1 releases Jun 5, 2024
@reginapizza reginapizza force-pushed the rebase-master-0.10.0 branch from 907f49f to 90a88d7 Compare June 5, 2024 17:21
@reginapizza
Copy link
Collaborator Author

fyi- just updated this to also included the OLM manifests for the 0.9.1 release as well since those were also missing from master

@anandf
Copy link
Collaborator

anandf commented Jun 6, 2024

Generated Code step is failing as there is diff reported in make bundle. @reginapizza Can you please take a look ?

@reginapizza
Copy link
Collaborator Author

@jgwest can you please re-review and remove requested changes if it looks okay now? There's currently a block on merging with it. Thank you!

Copy link
Member

@jgwest jgwest left a comment

Choose a reason for hiding this comment

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

LGTM, thanks @reginapizza!

@iam-veeramalla iam-veeramalla merged commit 22ded96 into argoproj-labs:master Jun 6, 2024
7 checks passed
anandf pushed a commit to anandf/argocd-operator that referenced this pull request Jun 6, 2024
* argocd-operator v0.10.0 release (argoproj-labs#1343)

* fix typo in metadata (argoproj-labs#1354)

* update replaces version number for 0.10.0 to 0.9.1 (argoproj-labs#1365)

* update make deploy manifests

* rebase with 0.9.1

* fix bundle manifest control plane

* make bundle

Signed-off-by: Anand Francis Joseph <[email protected]>
anandf added a commit that referenced this pull request Jun 7, 2024
* fix: don't change the default policy to reencrypt if the TLS secret is present (#1401)

Signed-off-by: Chetan Banavikalmutt <[email protected]>
Signed-off-by: Anand Francis Joseph <[email protected]>

* Add SeccompProfile for PodSecurityStandards "restricted" to avoid Pod Security Violations on restricted namespaces (#1288)

* Add SeccompProfile for PSA restricted

It is necessary to set the seccompProfile to RuntimeDefault to meet the
PodSecurityStandards of restricted, which will be a common default in
clusters.
Otherwise the workloads will be evaluated as baseline and rejected, if
not set otherwise in the namespace metadata.

* fix: failing test

Signed-off-by: iam-veeramalla <[email protected]>

---------

Signed-off-by: iam-veeramalla <[email protected]>
Co-authored-by: iam-veeramalla <[email protected]>
Signed-off-by: Anand Francis Joseph <[email protected]>

* Rebase master with 0.10.0 and 0.9.1 releases (#1403)

* argocd-operator v0.10.0 release (#1343)

* fix typo in metadata (#1354)

* update replaces version number for 0.10.0 to 0.9.1 (#1365)

* update make deploy manifests

* rebase with 0.9.1

* fix bundle manifest control plane

* make bundle

Signed-off-by: Anand Francis Joseph <[email protected]>

---------

Signed-off-by: Chetan Banavikalmutt <[email protected]>
Signed-off-by: Anand Francis Joseph <[email protected]>
Signed-off-by: iam-veeramalla <[email protected]>
Co-authored-by: Chetan Banavikalmutt <[email protected]>
Co-authored-by: Krzysztof Ostrowski <[email protected]>
Co-authored-by: iam-veeramalla <[email protected]>
Co-authored-by: Regina Scott <[email protected]>
anandf added a commit to anandf/argocd-operator that referenced this pull request Jun 7, 2024
* fix: don't change the default policy to reencrypt if the TLS secret is present (argoproj-labs#1401)

Signed-off-by: Chetan Banavikalmutt <[email protected]>
Signed-off-by: Anand Francis Joseph <[email protected]>

* Add SeccompProfile for PodSecurityStandards "restricted" to avoid Pod Security Violations on restricted namespaces (argoproj-labs#1288)

* Add SeccompProfile for PSA restricted

It is necessary to set the seccompProfile to RuntimeDefault to meet the
PodSecurityStandards of restricted, which will be a common default in
clusters.
Otherwise the workloads will be evaluated as baseline and rejected, if
not set otherwise in the namespace metadata.

* fix: failing test

Signed-off-by: iam-veeramalla <[email protected]>

---------

Signed-off-by: iam-veeramalla <[email protected]>
Co-authored-by: iam-veeramalla <[email protected]>
Signed-off-by: Anand Francis Joseph <[email protected]>

* Rebase master with 0.10.0 and 0.9.1 releases (argoproj-labs#1403)

* argocd-operator v0.10.0 release (argoproj-labs#1343)

* fix typo in metadata (argoproj-labs#1354)

* update replaces version number for 0.10.0 to 0.9.1 (argoproj-labs#1365)

* update make deploy manifests

* rebase with 0.9.1

* fix bundle manifest control plane

* make bundle

Signed-off-by: Anand Francis Joseph <[email protected]>

---------

Signed-off-by: Chetan Banavikalmutt <[email protected]>
Signed-off-by: Anand Francis Joseph <[email protected]>
Signed-off-by: iam-veeramalla <[email protected]>
Co-authored-by: Chetan Banavikalmutt <[email protected]>
Co-authored-by: Krzysztof Ostrowski <[email protected]>
Co-authored-by: iam-veeramalla <[email protected]>
Co-authored-by: Regina Scott <[email protected]>
Signed-off-by: Anand Francis Joseph <[email protected]>
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.

4 participants