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

Clean up versions.sh #1076

Merged
merged 1 commit into from Dec 16, 2015
Merged

Clean up versions.sh #1076

merged 1 commit into from Dec 16, 2015

Conversation

sdodson
Copy link
Member

@sdodson sdodson commented Dec 16, 2015

No description provided.

@sdodson
Copy link
Member Author

sdodson commented Dec 16, 2015

Another possible fix for #1072

@abutcher
Copy link
Member

Not sure if this difference will matter yet...

New:
[root@master4 ~]# sh versions.sh atomic-openshift openshift
---
curr_version: 3.1.0.4-1.git.15.5e061c3.el7aos 
avail_version: 3.0.2.0-0.git.38.7576bc5.el7ose 

Old:
[root@master4 ~]# sh ~/openshift-ansible/playbooks/common/openshift-cluster/upgrades/files/versions.sh atomic-openshift openshift
---
curr_version: 3.1.0.4-1.git.15.5e061c3.el7aos 
avail_version:

Un-upgraded looks good though

New:
[root@master4 ~]# sh versions.sh atomic-openshift openshift
---
curr_version: 3.0.2.0-0.git.38.7576bc5.el7ose 
avail_version: 3.1.0.4-1.git.15.5e061c3.el7aos 

Old:
[root@master4 ~]# sh ~/openshift-ansible/playbooks/common/openshift-cluster/upgrades/files/versions.sh atomic-openshift openshift
---
curr_version: 3.0.2.0-0.git.38.7576bc5.el7ose 
avail_version: 3.1.0.4-1.git.15.5e061c3.el7aos

@sdodson
Copy link
Member Author

sdodson commented Dec 16, 2015

Added the grep -v el7ose back in.

@abutcher
Copy link
Member

lgtm

@sdodson
Copy link
Member Author

sdodson commented Dec 16, 2015

@twiest i think this is ready to merge

@detiber
Copy link
Contributor

detiber commented Dec 16, 2015

👍

twiest added a commit that referenced this pull request Dec 16, 2015
@twiest twiest merged commit 2200cf2 into openshift:master Dec 16, 2015
@sdodson sdodson deleted the versions branch July 6, 2016 13:49
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.

None yet

4 participants