Skip to content
This repository has been archived by the owner on Dec 8, 2020. It is now read-only.

merge changes for 6.3 in master. #52

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

merge changes for 6.3 in master. #52

Show file tree
Hide file tree
Changes from 4 commits
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
06812f6
Updated to work with Satellite 6.3 which is now automatically updatin…
gearboxscott Apr 27, 2018
e298e14
added comments about using cvmanager with Satellite 6.3.
gearboxscott Apr 27, 2018
375c7a4
add comment about this version will not work with 6.2 or less.
gearboxscott Apr 27, 2018
54d2a87
add origin to show where the original code came from.
gearboxscott Apr 27, 2018
361bd77
re-uploaded the file
gearboxscott Apr 27, 2018
7572c36
changed ruby to tfm-ruby
gearboxscott May 1, 2018
7576a29
removed tfm from ruby
gearboxscott May 2, 2018
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
14 changes: 14 additions & 0 deletions README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,3 +1,17 @@
# Satellite 6.3 Update

Satellite 6.3 no longer provide apipie via the repositories. To use this utility, install apipie.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not completely true, we just don't provide apipie-bindings for the system ruby.

I think we should document the various ways users can run it, and let them decide:

  1. the most Ruby way would be bundle:
bundle install
bundle exec ruby ./cvmanager …
  1. scl enable tfm -- ./cvmanager works on any production install of Katello/Satellite
  2. gem install apipie-bindings followed by a ./cvmanager also works, but is the most unclean one, imho


```
gem install apipie
```

This version will no longer work with Satellite 6.2 or less.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why not to make an API call to check if latest is supported or not?
So the code may be backward compatible.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Rocco83, when @mccun934 told me to merge the changes into the current code for 6.3 and they can download a prior release for 6.2 or less. We can add this functionality but then we need to add in the update section of the code a branch for 6.2 or less. @mccun934 do you me to do that or have them download a prior release?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

How big is the diff between 6.2 and 6.3 compat? I'd prefer to remain 6.2 compatibility, but if it's too much of a burden, I'd tag a last 6.2 compatible release and carry on with the modern code.


# Origin

This was forked from katello-cvmanger https://github.com/RedHatSatellite/katello-cvmanager and updated to work with Satellite 6.3.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is not a fork, it's a tribute, erm, PR ;)


# cvmanager

For automation of some common tasks related to Content Views we created a tool called `cvmanager`. It consists of a Ruby script (`cvmanager`) and a YAML-formatted configuration file (`cvmanager.yaml`). The various features are described in the following chapters.
Expand Down
Loading