Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Some time the past few weeks, the testbot stopped downloading media's hard dependencies (file_entity or views)
Maybe a dependency rebuild would solve this?
https://qa.drupal.org/pifr/test/158824
Media's info file.
http://drupalcode.org/project/media.git/blob/5839787d839fe880d9334d3f4c3...
even weird: the alpha2 version test download the dependencies normally
https://qa.drupal.org/pifr/test/607848
Comments
Comment #1
Dave ReidI filed #2067261: Rebuild dependencies for Media (missing File entity 2.x dependency) a while ago but it appears it did not resolve the issue. This is at least the third time the testbot has regressed for us. :/ (#1802156)
Comment #2
rfayThe dependencies were in fact completely empty, now after a rebuild:
Comment #3
jthorson CreditAttribution: jthorson commenteds/testbot/project_dependency. The testbots cause enough problems on their own that they don't need to assume credit for issues caused elsewhere. ;)
Dependencies rebuilt. Do these look correct?
EDIT: Beat to the punch. Removing duplicate array text.
Comment #4
jthorson CreditAttribution: jthorson commentedCrosspost
Comment #5
ParisLiakos CreditAttribution: ParisLiakos commentedback to green:)
thanks a lot that was quick!
Comment #7
ParisLiakos CreditAttribution: ParisLiakos commentedcan we have a dependency rebuild again please? i have added token module as test_dependency a few days now (e7be30f), but bot doesnt check it out. I have no clue how to trigger a rebuild manually. shouldnt a new commit trigger it? or the fact the a new .dev release was packaged?
Comment #8
ParisLiakos CreditAttribution: ParisLiakos commentedlet me know if i should open a new issue
Comment #9
rfayYou really should open a new issue for a new event.
I took a look at this and after rebuild media's dependencies come out empty. None at all.
I have no idea why that would be.
Comment #10
rfayDo you *really* depend on token module for tests? It's still just a UI right? Not that I can see anything wrong with what you've done.
Comment #11
ParisLiakos CreditAttribution: ParisLiakos commentedits more than UI unfortunately. it provides field tokens that core does not. And we have a usecase we need to test eg that fields attached to a file entity can be overriden in WYSIWYG and the overrides display correctly.
related issue #2116451: Add test coverage for JSON placeholder rendering
I introduce a test there, which has token as dependency, but it never runs unfortunately. i am quite stuck :/
thanks for your time
Comment #12
rfayI'm unable to recreate this on qa6-drupal. The dependency build works fine. I'm going to have to rebuild that thing and see what might be different about the db.
Comment #13
rfayI'm unable to recreate this on a local machine with a current drupal.org database either (and a current checkout from bzr).
Also, drush pdsd shows the correct values when I dump the project_dependency tables off of util and load them onto my local.
Suggestions of what could be different on production are very welcome. Drush5 hasn't *changed* over there. It *appears* to go through the whole process successfully.
Comment #14
rfayHere is the actual situation on util on the live db.
So it looks to me like the problem is somewhere in the calculation, not the creation of the data in the db. But it's something local to the d.o environment. Probably doesn't have to do with drush pdsd, but rather the things it calls.
Comment #15
rfayIt seems to me like it's a waste to work on this right now with the D7 push in 2 days. I'm going to leave it alone until after the upgrade, and we'll see what happens then. Fire and brimstone, I imagine.
If you don't see me pick it up by next week, ping me here.
Comment #16
ParisLiakos CreditAttribution: ParisLiakos commentedthanks! i agree, D7 upgrade is far more important. good luck!
Comment #17
jthorson CreditAttribution: jthorson commentedFor what it's worth, here's the current output on the Drupal.org D7 test site.
That said, I don't know how recent the data in the table would be.
Comment #18
rfayIf you have a chance (and have permissions). please do a
drush pdpp media
so that it updates, and then do thedrush pdsd media 7.x-2.x-dev
again. That will get completely current data. The pdpp must be done as the appropriate user...Comment #19
jthorson CreditAttribution: jthorson commented7.x version looks good!
Comment #20
ParisLiakos CreditAttribution: ParisLiakos commentedthanks but seems that somehow it only checkouts media again:
https://qa.drupal.org/pifr/test/158824 :/
Comment #21
rfay@ParisLiakos, Drupal.org is still on 6.x. It goes to 7 tomorrow. The hope (bolstered by @jthorson) is that whatever this oddity is, it will go away on 7.x.
Comment #22
ParisLiakos CreditAttribution: ParisLiakos commentedah! sorry, i thought #19 referred to the 6.x version ;)
i should pay more attention
Comment #23
rfayI rebuilt today and it looks good to me (on D7). Hopefully it will stick this time. Given the number of times this has happened in the past, it's my bet there's some obscure bug that only gets reported on media. So it's possible we'll see it again.
Comment #24
rfayComment #25
rfayOops, since adding token as test_dependency and it seems not to show token as a dependency, now we have a different problem.
Comment #26
jthorson CreditAttribution: jthorson commentedI assumed 'dependency_type' => 1 was a test_dependency ... is that not true?
Comment #27
ParisLiakos CreditAttribution: ParisLiakos commentedthanks, seems all dependencies check out as should now.
and yes token is indeed a test_dependency, not a normal one
Comment #28
rfayI was just hallucinating. Returned to this and it looked good.