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

Grains item patch #23437

Closed
wants to merge 2 commits into from
Closed

Conversation

cedwards
Copy link
Contributor

@cedwards cedwards commented May 7, 2015

Updating grains.item to support querying nested grains.

@jfindlay
Copy link
Contributor

jfindlay commented May 7, 2015

@cedwards, this has got your other pull request in it.

@jfindlay
Copy link
Contributor

jfindlay commented May 7, 2015

The grains fix here needs to go into 2014.7.

@jfindlay jfindlay added the pending-changes The pull request needs additional changes before it can be merged label May 7, 2015
@rallytime
Copy link
Contributor

@cedwards It looks like your Apache Traffic Server module was accidentally included in this pull request, and was already merged into develop in #23436. I grabbed the grains fix commit and separated it out from this pull request in #23437 to keep the code cleaner and separate. That grains fix will be merged into develop, so I am going to close this PR in favor of that one. Let me know if I messed anything! :)

@rallytime rallytime closed this May 14, 2015
thatch45 added a commit that referenced this pull request May 14, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pending-changes The pull request needs additional changes before it can be merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants