forked from openshift/oadp-operator
-
Notifications
You must be signed in to change notification settings - Fork 0
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
Fork Sync: Update from parent repository #103
Open
kaovilai
wants to merge
285
commits into
kaovilai:master
Choose a base branch
from
openshift:master
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
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
…#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.
…datamover config map (#1078)
* start troubleshooting doc update * update the troubleshotting doc * Update TROUBLESHOOTING.md update from scott's comments. * add details * s/backup/restore * Update docs/TROUBLESHOOTING.md use deploy vs. deployment.apps Co-authored-by: Tiger Kaovilai <[email protected]> --------- Co-authored-by: Tiger Kaovilai <[email protected]>
* wip cheat sheet * WIP cheat sheet * fix code block w/ yaml tag and other nits * ugh, what's wrong with me * Update oadp_cheat_sheat.md Remove hackmd note, and align the introduction to other cheat sheets. * Update docs/oadp_cheat_sheat.md Co-authored-by: Tiger Kaovilai <[email protected]> * Update oadp_cheat_sheat.md include a warning and context about oc delete $datamover_objects * Update oadp_cheat_sheat.md udpate based on scott's review * backup_name * color code * Update oadp_cheat_sheat.md typo * Update docs/oadp_cheat_sheat.md * Update docs/oadp_cheat_sheat.md --------- Co-authored-by: HackMD <37423+hackmd-hub[bot]@users.noreply.github.com> Co-authored-by: Tiger Kaovilai <[email protected]>
* first pass at restic troubleshooting * Update restic_troubleshooting.md clean up * Update restic_troubleshooting.md fix clients * Update restic_troubleshooting.md Update based on @mateusoliveira43 feedback, remove url's and bucket names. * Update restic_troubleshooting.md Thanks Scott! * Update restic_troubleshooting.md update for Scott and Shubham's comment * get restic secret in one line * Update heading w/ context.
Nice reference to have while the official doc is in flux
* Add volsync cron expression support * update schedule paramter description
* fix typo in cheet_sheet file name * another typo in url
Documentation for the OADP to use User Workload Monitoring and sample Alerting Rule. Depends-On: #1081 Fixes: #769 https://issues.redhat.com/browse/OADP-1887 https://issues.redhat.com/browse/OADP-661 Signed-off-by: Michal Pryc <[email protected]>
* update the datamover blog for 1.2.0 * Update data-mover-1.2.0-overview.md reformat * Update data-mover-1.2.0-overview.md meh update * Updated diagrams * Update image src * fix: update few details * Suggestion from last scrum * rearrange vsb/vsr feilds, update volsync repo * Update data-mover-1.2.0-overview.md added roadmap * Update blogs/data-mover/data-mover-1.2.0-overview.md Co-authored-by: Tiger Kaovilai <[email protected]> * Update blogs/data-mover/data-mover-1.2.0-overview.md Co-authored-by: Tiger Kaovilai <[email protected]> * updating for the final review --------- Co-authored-by: hhpatel14 <[email protected]> Co-authored-by: Tiger Kaovilai <[email protected]>
* point to 1.2 branch for velero and csi plugin on master * ci-Dockerfile: `chmod -R 777 $(go env GOPATH)` * add `/opt/app-root/src/go/bin` to GOPATH to expose ginkgo CLI Signed-off-by: Tiger Kaovilai <[email protected]> * e2e: ignore level=debug logs containing "error" --------- Signed-off-by: Tiger Kaovilai <[email protected]> Co-authored-by: Tiger Kaovilai <[email protected]>
…the Velero deployment. (#1081) First change to remove cluster monitoring of OADP that will be replaced by the user workload monitoring (UWM). Related Issues: #769 https://issues.redhat.com/browse/OADP-1887 https://issues.redhat.com/browse/OADP-661 The openshift-adp-velero-metrics-svc is left to easy process of enabling UWM. Once UWM is enabled, it will require setting up ServiceMonitor and configuring alerts or dashboards that are crucial for a particular use-case. Enablement of user workload monitoring with additional documentation is not part of this PR to easilly re-add the cluster monitoring in the future by reverting this change. Signed-off-by: Michal Pryc <[email protected]>
* Fix typo in CRD description * run `make bundle` --------- Co-authored-by: Tiger Kaovilai <[email protected]>
New configuration for the file system backup & restore design that will be used by the OADP and allow to choose Restic or Kopia uploader type. _template.md borrowed from the Upstream vmware-tanzu/velero/main/design/_template.md Signed-off-by: Michal Pryc <[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]>
This is because docker build has become aliased to docker buildx build whose default no longer load build result to docker images which mean subsequent call to `docker-push` target fails. Signed-off-by: Tiger Kaovilai <[email protected]>
Signed-off-by: Mateus Oliveira <[email protected]>
…1596) * add doc for automount_service_account_token exception * address PR feedback
Small CRD updates to make the following PR passing: migtools/oadp-non-admin#130 Signed-off-by: Michal Pryc <[email protected]>
Signed-off-by: Mateus Oliveira <[email protected]>
CRD updates for the following non-admin PR: migtools/oadp-non-admin#128 Signed-off-by: Michal Pryc <[email protected]>
Signed-off-by: Tiger Kaovilai <[email protected]>
Signed-off-by: Tiger Kaovilai <[email protected]>
Signed-off-by: Tiger Kaovilai <[email protected]>
* feat: Show more DPA info with 'oc get' Signed-off-by: Mateus Oliveira <[email protected]> * fixup! feat: Show more DPA info with 'oc get' Signed-off-by: Mateus Oliveira <[email protected]> --------- Signed-off-by: Mateus Oliveira <[email protected]>
* fix: Operator SDK update guidelines Signed-off-by: Mateus Oliveira <[email protected]> * fixup! fix: Operator SDK update guidelines Signed-off-by: Mateus Oliveira <[email protected]> --------- Signed-off-by: Mateus Oliveira <[email protected]>
With removal of forceBackup option from the Non Admin Backup spec it's necessary to update manifests. Signed-off-by: Michal Pryc <[email protected]>
* Update manifests to match non-admin PR139 Updated manifests for the NonAdminBackupStorageLocation Signed-off-by: Michal Pryc <[email protected]> * Update description and display for the NonAdminBackupStorageLocation Sync manifests and update description and display for hte NonAdminBackupStorageLocation. Signed-off-by: Michal Pryc <[email protected]> --------- Signed-off-by: Michal Pryc <[email protected]>
…re (#1621) Setting the CLUSTER_TYPE, CLUSTER_OS and CLUSTER_ARCH in the Makefile happened after it was used in the Makefile making them effectively unset. Moving the part to discover those variables above their usage fixes the problem of images being built for improper arch. Signed-off-by: Michal Pryc <[email protected]>
* Remove multiple single quotes from message field * Fix unit test
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.