Fork Sync: Update from parent repository #103
Open
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* OADP-148 Replace velero pod exec with local velero. Signed-off-by: Matthew Arnold <[email protected]> * Assorted small fixes for must-gather. Fill out git commit in velero client version, fix DPA version gathering, and fix a mkdir problem in log backup. Signed-off-by: Matthew Arnold <[email protected]> --------- Signed-off-by: Matthew Arnold <[email protected]>
…is false (#1012) * Validation to ensure vsm plugin is removed from DPA if DM is false * update validation error message
* update documentation for datamover and docs/faq * shallow-copy only impacts backup times
Signed-off-by: Prasad Joshi <[email protected]>
* oadp-1.1: OADP-1557 Add all velero server args supported to DPA (#987) * refactor: remove reconciler from functions Signed-off-by: Tiger Kaovilai <[email protected]> * oadp-1.1: go get openshift/velero:oadp-1.1 Signed-off-by: Tiger Kaovilai <[email protected]> * add server package for velero server options Signed-off-by: Tiger Kaovilai <[email protected]> * wip Signed-off-by: Tiger Kaovilai <[email protected]> * replace apimachinery to avoid velero update Signed-off-by: Tiger Kaovilai <[email protected]> * Delete util.go * use `base := make(map[T]T)` * Update go.mod, remove todo comment. * Revert "use `base := make(map[T]T)`" This reverts commit 0b9042a. * controllers: add debugMode Signed-off-by: Tiger Kaovilai <[email protected]> * add metrics port parsing error Signed-off-by: Tiger Kaovilai <[email protected]> * simplified range expression Signed-off-by: Tiger Kaovilai <[email protected]> * rely on remove duplicate at the end of the function Signed-off-by: Tiger Kaovilai <[email protected]> * Apply suggestions from code review --------- Signed-off-by: Tiger Kaovilai <[email protected]> * oadp-1.2 changes. Signed-off-by: Tiger Kaovilai <[email protected]> * Use 4h for fs-backup-timeout Signed-off-by: Tiger Kaovilai <[email protected]> * use oadp restic default timeout of 1h --------- Signed-off-by: Tiger Kaovilai <[email protected]>
Signed-off-by: Prasad Joshi <[email protected]>
* SC to datamover api * check for existing sc in cluster * bundle updates Co-authored-by: Emily McMullan <[email protected]> * Use typed strings for accessMode Signed-off-by: Tiger Kaovilai <[email protected]> * Add spec.features.datamover.storageclass to e2e with provider storageclass name. Signed-off-by: Tiger Kaovilai <[email protected]> * rename volumeoptions sc * update crds * Fix `ProviderStorageClassName()` relative path Signed-off-by: Tiger Kaovilai <[email protected]> --------- Signed-off-by: Tiger Kaovilai <[email protected]> Co-authored-by: Tiger Kaovilai <[email protected]>
privledged true was throwing errors at creation I also saw errors that caused the restore to fail.
* watch volumemover resources live * add directions * add restore example * update help, and sure help w/ no options
* OADP-1931: Update description in CSV (#1008) [OADP-1931](https://issues.redhat.com/browse/OADP-1931) Improve Cluster Service Version YAML's Description fields Incorporate feedback from [comment](https://issues.redhat.com/browse/OADP-1931?focusedId=22320719&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-22320719) * Revert 'oneliner' description
* Make deploy-olm work in master Bump openshift/velero to oadp-1.2 latest Makefile: add envtest to manifest remove `sigs.k8s.io/controller-runtime/pkg/envtest/printer` dependency `go get github.com/backube/volsync` go fmt bundle update. Signed-off-by: Tiger Kaovilai <[email protected]> * avoid ginkgo go mod updates --------- Signed-off-by: Tiger Kaovilai <[email protected]>
Signed-off-by: Tiger Kaovilai <[email protected]>
Without this, the velero command is not able to find individual backups or restores when collecting logs. Signed-off-by: Matthew Arnold <[email protected]> Co-authored-by: Matthew Arnold <[email protected]>
* add samples for cephfs/rbd for documentation This is NOT intended to be executed in CI * added custom storage class manifests * added pv manifects * eeek param backingSnapshot was removed * add OG storage classes for my love * wes IS dumb
* add doc with datamover advanced volumeoptions example * update from suggestions
…#1041) * must-gather suite test Signed-off-by: Tiger Kaovilai <[email protected]> * add must-gather validation function to suite brCase Signed-off-by: Tiger Kaovilai <[email protected]> * Fix dpa name expected to match instance name Signed-off-by: Tiger Kaovilai <[email protected]> --------- Signed-off-by: Tiger Kaovilai <[email protected]>
* [WIP] Make data mover dummy pod configurable Signed-off-by: Jason Montleon <[email protected]> * fix datamover_test for dummy pod image env * update config/manager/manager.yaml Signed-off-by: Tiger Kaovilai <[email protected]> --------- Signed-off-by: Jason Montleon <[email protected]> Signed-off-by: Tiger Kaovilai <[email protected]> Co-authored-by: Tiger Kaovilai <[email protected]>
…1068) * Add velero replica count override for debugging. Signed-off-by: Matthew Arnold <[email protected]> * Add datamover replica count override for debugging. Signed-off-by: Matthew Arnold <[email protected]> * Add unit test for Velero replica count override. Signed-off-by: Matthew Arnold <[email protected]> * Add unit test for datamover replica count override. Signed-off-by: Matthew Arnold <[email protected]> --------- Signed-off-by: Matthew Arnold <[email protected]>
* WIP: first pass at non-admin tekton backup * remove old files * remove cruft * got it all working, need to retest * setup almost working again with templates * fix namespace for image import * Update README.md add screenshot and logs * update the path where we list the backup cr * add steps to check status and finally * add script to create htpasswd oauth * Update README.md latest updates * Update README.md fix typo * macos compatible sed * ensure sed is updating all entries * add a prereq check and bail if we're overwriting oauth config * Update README.md readme updates * Update README.md provide links to doc * fix up prereq script * Update README.md further updates * Update README.md provide more context and technical details * Update README.md add note about installing multiple versions of OADP and FAQ * Update README.md fix typo * Update README.md typo * Update README.md typo * Update README.md add arch diagram * Update README.md fix nit * Update README.md updated diagram * add restore pipeline * restore cr needs a backupname * nit on the restore status * Update README.md update docs w/ restore * Update README.md added overall pipeline view * Update README.md pvc warning * refactor user creation, clean up * more refactor * Update README.md updates for latest changes. workarounds for demo_users.sh and how to verify it. * Update README.md added pvc issue * Update README.md update to use a volumeclaimtemplate * build a container w/ b/r cr's on it * simplify pipeline * include the RESTORE_NAME in templates * add check to see if user has access to provided namespace * put in restore checks * turn off shell debug * break out image import to be one time operation * create an advanced backup cr example * clean up scripts and README * update the README * Update README.md update screenshots <NOT COMPLETE> * fix typo * Update README.md fix screenshots * sed in-place command fix for macos Signed-off-by: Tiger Kaovilai <[email protected]> * Updated Readme with improved screenshots, new diagrams. Rework of check_prerequisites.sh script to check and then print summary of all errors. Signed-off-by: Michal Pryc <[email protected]> * Update README.md * Update README.md * add tigers diagram * Update README.md * move new diagram * move tekton demo to blog section of repo * Update README.md spelling typo * fix spelling * Update README.md --------- Signed-off-by: Tiger Kaovilai <[email protected]> Signed-off-by: Michal Pryc <[email protected]> Co-authored-by: Tiger Kaovilai <[email protected]> Co-authored-by: Michal Pryc <[email protected]>
ensure Tekton is capitalized consistently.
Signed-off-by: Wesley Hayutin <[email protected]>
* not working Signed-off-by: Wesley Hayutin <[email protected]> * WOOT, working w/ make test-e2e GINKGO_ARGS=--ginkgo.focus='MySQL application CSI' Signed-off-by: Wesley Hayutin <[email protected]> * all dpa creation / tests working Signed-off-by: Wesley Hayutin <[email protected]> * update other cloud providers w/ args Signed-off-by: Wesley Hayutin <[email protected]> * add note to docs Signed-off-by: Wesley Hayutin <[email protected]> * address comments and fix aws settings * updated for Mateus's comments * removed image override test * remove unsed var declaration * other mindor changes Signed-off-by: Wesley Hayutin <[email protected]> * remove csi plugin override Signed-off-by: Wesley Hayutin <[email protected]> * remove unrequired deepcopy Signed-off-by: Wesley Hayutin <[email protected]> --------- Signed-off-by: Wesley Hayutin <[email protected]>
* fix: Default permission for Secrets Signed-off-by: Mateus Oliveira <[email protected]> * fixup! fix: Default permission for Secrets Signed-off-by: Mateus Oliveira <[email protected]> * fixup! fix: Default permission for Secrets Signed-off-by: Mateus Oliveira <[email protected]> * fixup! fix: Default permission for Secrets Signed-off-by: Mateus Oliveira <[email protected]> --------- Signed-off-by: Mateus Oliveira <[email protected]>
* doc: How to create new release branch CI files Signed-off-by: Mateus Oliveira <[email protected]> * fixup! doc: How to create new release branch CI files Signed-off-by: Mateus Oliveira <[email protected]> * fixup! doc: How to create new release branch CI files Signed-off-by: Mateus Oliveira <[email protected]> * fixup! doc: How to create new release branch CI files Signed-off-by: Mateus Oliveira <[email protected]> * fixup! doc: How to create new release branch CI files Signed-off-by: Mateus Oliveira <[email protected]> * fixup! doc: How to create new release branch CI files Signed-off-by: Mateus Oliveira <[email protected]> * fixup! doc: How to create new release branch CI files Signed-off-by: Mateus Oliveira <[email protected]> --------- Signed-off-by: Mateus Oliveira <[email protected]>
Signed-off-by: Mateus Oliveira <[email protected]>
* fix: Add table for badges in README Signed-off-by: Mateus Oliveira <[email protected]> * fixup! fix: Add table for badges in README Signed-off-by: Mateus Oliveira <[email protected]> --------- Signed-off-by: Mateus Oliveira <[email protected]>
* Add option for upstream virtualization operator. Use this to tweak the subscription settings and some expected namespaces. Signed-off-by: Matthew Arnold <[email protected]> * Do not fail test on missing images namespace. Signed-off-by: Matthew Arnold <[email protected]> * Avoid "cannot configure to watch own namespace". Signed-off-by: Matthew Arnold <[email protected]> * Add DataVolume clone permissions for cirros-test. Signed-off-by: Matthew Arnold <[email protected]> * Remove permissions for kubevirt-os-images. Going to try to keep everything in openshift-virtualization-os-images namespace instead. Installing this CRD as-is fails because kubevirt-os-images does not exist yet. Signed-off-by: Matthew Arnold <[email protected]> * Force upstream test run. Signed-off-by: Matthew Arnold <[email protected]> * Copy fedora data source to os-images namespace. The community KubeVirt operator puts these in "kubevirt-os-images", but to keep fixed YAML templates the test needs them in "openshift-virtualization-os-images". This adds a way to copy an existing data source to a new namespace. Signed-off-by: Matthew Arnold <[email protected]> * Turn off upstream-only CI. Undo temporary change to force community operator in tests. Signed-off-by: Matthew Arnold <[email protected]> * Don't fail if test storage classes already exist. Signed-off-by: Matthew Arnold <[email protected]> * Explicitly set Fedora volume mode for test. Trying to work around image clone failure in CI. Signed-off-by: Matthew Arnold <[email protected]> * Revert "Explicitly set Fedora volume mode for test." This reverts commit a663e29. Signed-off-by: Matthew Arnold <[email protected]> --------- Signed-off-by: Matthew Arnold <[email protected]>
* convert the troubleshotting.md to proper md * mostly just converting to markdown * moved some links around, imho better organization. * not perfect but better Signed-off-by: Wesley Hayutin <[email protected]> * update faq link --------- Signed-off-by: Wesley Hayutin <[email protected]>
* OADP-5024 Virtualization troubleshooting document. Start this off with label selectors for individual VMs. Signed-off-by: Matthew Arnold <[email protected]> * Add syntax highlighting. Signed-off-by: Matthew Arnold <[email protected]> --------- Signed-off-by: Matthew Arnold <[email protected]>
Signed-off-by: Wesley Hayutin <[email protected]>
…es (#1582) The Makefile auto-discovers the build tool, defaulting to docker or podman if available. Users can override this by setting the CONTAINER_TOOL environment variable to specify their preferred tool. This approach is aligned with other operators, such as non-admin, while also adding automatic tool discovery for improved usability and transparency. Signed-off-by: Michal Pryc <[email protected]>
Signed-off-by: Mateus Oliveira <[email protected]>
Update of the manifests after merging: migtools/oadp-non-admin#101 Signed-off-by: Michal Pryc <[email protected]>
* some notes on schedules * updates from the reviews
* Add contributing.md * updated override info
Updates to the manifests required by the NAB PR: migtools/oadp-non-admin#108 Signed-off-by: Michal Pryc <[email protected]>
* add a first pass at kopia maintenance debug info * add delete updates and restic * updates after review * fix after proof read * update from michal's comments
Signed-off-by: Mateus Oliveira <[email protected]>
* fix: Enforce NonAdminBackup spec field values Signed-off-by: Mateus Oliveira <[email protected]> * fixup! fix: Enforce NonAdminBackup spec field values rework of logic Signed-off-by: Mateus Oliveira <[email protected]> * fixup! fix: Enforce NonAdminBackup spec field values fix nil pointer Signed-off-by: Mateus Oliveira <[email protected]> * fixup! fix: Enforce NonAdminBackup spec field values fix comparison check Signed-off-by: Mateus Oliveira <[email protected]> * fixup! fix: Enforce NonAdminBackup spec field values make update-non-admin-manifests Signed-off-by: Mateus Oliveira <[email protected]> * fixup! fix: Enforce NonAdminBackup spec field values run make update-non-admin-manifests Signed-off-by: Mateus Oliveira <[email protected]> * fixup! fix: Enforce NonAdminBackup spec field values Signed-off-by: Mateus Oliveira <[email protected]> --------- Signed-off-by: Mateus Oliveira <[email protected]>
* fix: NAC install validation Signed-off-by: Mateus Oliveira <[email protected]> * fixup! fix: NAC install validation improve error messages Signed-off-by: Mateus Oliveira <[email protected]> * fixup! fix: NAC install validation add tests Signed-off-by: Mateus Oliveira <[email protected]> * fixup! fix: NAC install validation working example Signed-off-by: Mateus Oliveira <[email protected]> * fixup! fix: NAC install validation use another client Signed-off-by: Mateus Oliveira <[email protected]> * fixup! fix: NAC install validation validate first Signed-off-by: Mateus Oliveira <[email protected]> * fixup! fix: NAC install validation only error one DPA Signed-off-by: Mateus Oliveira <[email protected]> --------- Signed-off-by: Mateus Oliveira <[email protected]>
…in terminationMessagePolicy (#1590) * Fix OADP-5183: Ensure containers are all using FallbackToLogsOnError in terminationMessagePolicy * minor update
…1598) Update Non Admin manifests to include information about estimated queue position of the NAB objects. Signed-off-by: Michal Pryc <[email protected]>
Signed-off-by: Mateus Oliveira <[email protected]>
Signed-off-by: Mateus Oliveira <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.