#development
Title
h
hundreds-father-404
05/15/2019, 11:02 PMUnrelated to memory issue, would be much appreciated if someone could please take a look at https://github.com/pantsbuild/pants/pull/7722#discussion_r284483847. Rule is saying it’s not available when calling
TestBase.scheduler.product_request()./pants --no-v1 --v2 testw
witty-crayon-22786
05/15/2019, 11:26 PMthat error is a rule graph construction failure, which happens at graph construction time
in this case it probably means that the rule is not installed in that context
it's almost certainly related to this: https://github.com/pantsbuild/pants/blob/096b135b0b19f9425d97b1f77afb521f5b8584b5/src/python/pants/init/engine_initializer.py#L358-L359
👍 1
h
hundreds-father-404
05/15/2019, 11:28 PMOhh cool okay, i’ll add that line. Thanks!
w
witty-crayon-22786
05/15/2019, 11:28 PM...which i started fixing in https://github.com/pantsbuild/pants/pull/7661
will see if i can get that green tonight.
❤️ 1
h
hundreds-father-404
05/15/2019, 11:44 PMIt worked 🙂 but now this error:
Copy code
E Exception: No installed @rules can satisfy Select(InitInjectedDigest) for input Params(Snapshot).def rules()w
witty-crayon-22786
05/15/2019, 11:53 PMwhat kind of test class?
h
hundreds-father-404
05/15/2019, 11:53 PMTestBasew
witty-crayon-22786
05/15/2019, 11:54 PMTestBase consumes
def rulesBUT the new error is no longer a rule graph construction error: rather a runtime error
is there a RootRule for Snapshot?
doesn't look like it.
so: RootRules.
if a Parameter might be provided from "outside" the graph at the root of a request (ie
scheduler.product_request👍 1
(and if it won't, you shouldn't)
h
hundreds-father-404
05/15/2019, 11:57 PMwhere do I declare this? in src or the test file
w
witty-crayon-22786
05/15/2019, 11:57 PMif you're trying to expose the rule "for testing", then you might consider just declaring the
RootRule(Snapshot)def rulesa lot of things that are currently RootRules in the core should only actually be RootRules in tests (because we never request them via
product_requestback in a bit.
h
hundreds-father-404
05/16/2019, 12:02 AMYay! That was it
🎉 1