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

Running all gather scripts in background #91

Conversation

OdedViner
Copy link
Contributor

No description provided.

yati1998 and others added 29 commits August 8, 2023 12:26
…e-time

Export SINCE_TIME for use in sub scripts
This commit is to collect the events in
sorted order and also collect yamls and describe
of oc events resource

Signed-off-by: yati1998 <[email protected]>
This commit removes the sort-by flag from
oc describe events collection

Signed-off-by: yati1998 <[email protected]>
and add missing space after the namespace flag.

Signed-off-by: black-dragon74 <[email protected]>
*
The image pointed to did not exist.
*

Signed-off-by: Manish M Yathnalli <[email protected]>
Signed-off-by: black-dragon74 <[email protected]>
…s-esc

[gather-ns-resources] Delimit BASE_COLLECTION_PATH
Necessary changes required for ci-operator integration.
This patch should be merged before merging the one in the
openshift/release repo.

This patch does not modify any of the exisiting functionality.

Signed-off-by: black-dragon74 <[email protected]>
Signed-off-by: black-dragon74 <[email protected]>
…efac

[refrac] Use go build tools for building
Fix readme to point to correct must gather image
Signed-off-by: black-dragon74 <[email protected]>
…uild-ins

Add instructions to build locally
Note: You still need to export `ODF_MG_TOKEN`.

Example: `ODF_MG_TOKEN=<MY_TOKEN_HERE> make local`

This builds and tags the image as "odf-mg". You can override it
using `IMG_TAG` environment variable.

P.S: If using rancher, just export RANCHER variable in the ENV.

Extended example: `RANCHER=yes ODF_MG_TOKEN=<TOKEN_HERE> IMG_TAG=my-custom-tag make local`

Signed-off-by: black-dragon74 <[email protected]>
…l-build

[Makefile] Allow local builds by using `make local`
…l-build

[Doc] Update README for local builds using make
this commit sorts the oc describe and yaml events
according to last seen timestamp.

Signed-off-by: yati1998 <[email protected]>
Signed-off-by: mrudraia <[email protected]>

Signed-off-by: mrudraia <[email protected]>

Signed-off-by: mrudraia <[email protected]>

Signed-off-by: mrudraia <[email protected]>

Signed-off-by: mrudraia <[email protected]>

Signed-off-by: mrudraia <[email protected]>

Signed-off-by: mrudraia <[email protected]>

Signed-off-by: mrudraia <[email protected]>
…/cherry-pick-82-to-main

[main] Collect ODF Provider and Client resources
Copy link
Contributor

openshift-ci bot commented Dec 4, 2023

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: OdedViner
Once this PR has been reviewed and has the lgtm label, please assign yati1998 for approval. For more information see the Kubernetes Code Review Process.

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@yati1998
Copy link
Member

yati1998 commented Dec 5, 2023

@OdedViner it looks good it me, but can you confirm that the logs in gather-debug.log is also fine and in order? we can then merge the PR.

@yati1998
Copy link
Member

yati1998 commented Dec 6, 2023

@OdedViner I see you have added duplicate PR for this, can you close one of them.

@openshift-merge-robot
Copy link
Collaborator

PR needs rebase.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@OdedViner OdedViner changed the base branch from main to release-4.14 December 7, 2023 14:48
Copy link
Contributor

openshift-ci bot commented Dec 7, 2023

@OdedViner: No Bugzilla bug is referenced in the title of this pull request.
To reference a bug, add 'Bug XXX:' to the title of this pull request and request another bug refresh with /bugzilla refresh.

In response to this:

Running all gather scripts in background

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@OdedViner OdedViner closed this Dec 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants