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

Command to check Rancher dependencies for a RC #286

Merged
merged 29 commits into from
Nov 29, 2023

Conversation

johnatasr
Copy link
Contributor

@johnatasr johnatasr commented Oct 19, 2023

Proposed Changes

  • Export a md formatted file and check if repository by given commit contains dev- and rc tags in a sort of files
  • Added command line file
  • Added logic to validate all given files

Types of Changes

  • Command line

Testing

How to test:

1 - Compile ecm distro-tools with this branch make
2 - Repo with wrong deps:
Local

./cmd/rancher_release/bin/rancher_release-linux-amd64 check-rancher-rc-deps 

Remote

./cmd/rancher_release/bin/rancher_release-linux-amd64 check-rancher-rc-deps -c aa6b3fdb5b402b7ffbf0b7dfd506e9571c8759e5 

Example of exportation:

Linked Issues

#272

User-Facing Change

Further Comments

@johnatasr johnatasr changed the title Command to check dependencies for a final RC Command to check Rancher dependencies for a final RC Oct 19, 2023
@johnatasr johnatasr marked this pull request as ready for review October 19, 2023 23:04
Signed-off-by: Johnatas <[email protected]>
@johnatasr
Copy link
Contributor Author

Now the layout looks like:

image

Signed-off-by: Johnatas <[email protected]>
@briandowns
Copy link
Contributor

I don't think we should include the output in the form of a log message. A log message is to indicate some information or state of the application itself, not the data being displayed by the application. We should probably output the information without being in a log message.

@johnatasr johnatasr force-pushed the feat-racher-release-deps-check branch from de1ef07 to 01b8712 Compare November 17, 2023 15:17
Signed-off-by: Johnatas <[email protected]>
cmd/rancher_release/README.md Outdated Show resolved Hide resolved
cmd/rancher_release/check_rancher_rc_deps.go Outdated Show resolved Hide resolved
cmd/rancher_release/check_rancher_rc_deps.go Show resolved Hide resolved
repository/repository.go Outdated Show resolved Hide resolved
release/rancher/rancher.go Outdated Show resolved Hide resolved
release/rancher/rancher.go Outdated Show resolved Hide resolved
release/rancher/rancher.go Outdated Show resolved Hide resolved
repository/repository.go Outdated Show resolved Hide resolved
release/rancher/rancher.go Outdated Show resolved Hide resolved
release/rancher/rancher.go Outdated Show resolved Hide resolved
release/rancher/rancher.go Outdated Show resolved Hide resolved
Signed-off-by: Johnatas <[email protected]>
Signed-off-by: Johnatas <[email protected]>
release/rancher/rancher.go Outdated Show resolved Hide resolved
@johnatasr johnatasr requested a review from tashima42 November 28, 2023 13:49
@johnatasr johnatasr requested a review from brooksn November 28, 2023 17:15
@johnatasr johnatasr merged commit 0c7bfb6 into rancher:master Nov 29, 2023
2 checks passed
tashima42 pushed a commit to tashima42/ecm-distro-tools that referenced this pull request Dec 19, 2023
* change repo files

* func to check all deps and command

* add test

* last adjusts

* improve code

* change requires

* remove necessity of repo and org

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

* more improves

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

* more adjusts

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

* removing logrus

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

* adjust for message

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

* adjust for message

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

* add template tag

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

* add template

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

* add more changes

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

* using regex for rancher txt files

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

* add docs

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

* Update rancher.go

* remove unsed commit validation

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

* split dev deps

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

* update doc and requires

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

* remove unsed api url

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

* more fixes

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

* remove func contents

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

* more adjusts

* add line and content to min version

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

* add rancher images export in example

---------

Signed-off-by: Johnatas <[email protected]>
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.

4 participants