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

use nexus context to generate nexus url #50

Merged
merged 3 commits into from
Oct 30, 2013

Conversation

michaeld
Copy link

when the nexus context is modified, the Chef::Nexus.nexus_available is failing due to the incorrectly generated endpoint.

@@ -43,7 +43,22 @@ def get_nexus_data_bag(node)
#
# @return [Chef::Mash] the credentials entry in the data bag item
def get_credentials(node)
get_nexus_data_bag(node)[:credentials]
if Chef::Config[:solo]
Copy link
Contributor

Choose a reason for hiding this comment

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

This code was removed in 83edf35. Is there a reason it is coming back here? I created an issue a few days ago that needs to be addressed, to make this cookbook's usage a bit easier in Chef-solo (#48).

@KAllan357
Copy link
Contributor

Thanks for this! I am 👍 on this after the commented on code is removed. I'd like to see that code revisited in #48 so we can make this cookbook easier to use.

@michaeld
Copy link
Author

i only wanted the first commit. it automatically added my second commit to the pull request. let me see if i can get it back to the state i submitted it.

@michaeld
Copy link
Author

sorry about that. i only wanted the nexus context used when generating the url. i hope this will do.

@michaeld
Copy link
Author

do you want me to clean up the pull request sans the chef-solo branching and resubmit? I reverted that commit and the current state is just the nexus context url generator.

@KAllan357
Copy link
Contributor

Nope, this looks good! Thanks!

KAllan357 added a commit that referenced this pull request Oct 30, 2013
use nexus context to generate nexus url
@KAllan357 KAllan357 merged commit 5644460 into RiotGamesCookbooks:master Oct 30, 2013
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.

2 participants