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

Fork Sync: Update from parent repository #103

Open
wants to merge 285 commits into
base: master
Choose a base branch
from

Conversation

kaovilai
Copy link
Owner

No description provided.

kaovilai and others added 30 commits May 31, 2023 15:54
)

* add shallow copy params for datamover restore

* add note: cephfs shallow for restore is not supported
…#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.
* 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]>
mpryc and others added 30 commits November 12, 2024 11:33
* 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]>
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]>
* 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
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.