Skip to content

Conversation

@dsessler7
Copy link
Collaborator

Checklist:

  • Have you added an explanation of what your changes do and why you'd like them to be included?
  • Have you updated or added documentation for the change, as applicable?
  • Have you tested your changes on all related environments with successful results, as applicable?
    • Have you added automated tests?

Type of Changes:

  • New feature
  • Bug fix
  • Documentation
  • Testing enhancement
  • Other

What is the current behavior (link to any open issues here)?

Older installer before we moved the official installer to the postgres-operator repo

What is the new behavior (if this is a feature change)?

  • Breaking change (fix or feature that would cause existing functionality to change)

Latest installer with newer images and general installer structure.

Other Information:

@tjmoore4
Copy link
Contributor

tjmoore4 commented Jan 5, 2026

❓ Looks like the go-test failure is related to the CRD check failing. Do those need to be regenerated?

@dsessler7
Copy link
Collaborator Author

❓ Looks like the go-test failure is related to the CRD check failing. Do those need to be regenerated?

The issue is that the CRDs we put in our official installer are slightly different from the "raw" CRDs we generate in the check-generate make target. I think we need to have two sets of CRDs in the repo: official installer CRDs and in-development CRDs that are used by check-generate, deploy-dev and the like. I've created a Linear ticket for this issue.

Copy link
Contributor

Choose a reason for hiding this comment

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

❓ I'm a bit unclear as to what's happening with this license versus what we're doing with this one. Do we need both?

Copy link
Collaborator

Choose a reason for hiding this comment

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

The installer on main doesn't have this file but I do see it as a removed file from the examples repo. Maybe it should be added to main as well?

value: kustomize
- name: PGO_INSTALLER_ORIGIN
value: postgres-operator-repo
value: examples-repo
Copy link
Contributor

Choose a reason for hiding this comment

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

🔧 This value should probably be reverted.

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.

3 participants