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

Lack of cdnjs activity #13524

Closed
eastling opened this issue Oct 22, 2019 · 18 comments
Closed

Lack of cdnjs activity #13524

eastling opened this issue Oct 22, 2019 · 18 comments

Comments

@eastling
Copy link
Contributor

@eastling eastling commented Oct 22, 2019

Hello, it seems the last commit is in 17days ago. Why there is no commit recently?
image

@MattIPv4
Copy link
Member

@MattIPv4 MattIPv4 commented Oct 22, 2019

Long story super short:

  • Founder restricted access heavily so we (core maintainers) cannot do much at all
  • Founder disappeared off the face of the earth and we heard nothing
  • We (core maintainers) raised this internally and asked it be sorted
  • Nothing was sorted, so @PeterDaveHello has quit maintaining the project
    • As part of this, they appear to have disabled our updating automation (I disagree with this action, it harms the existing users of the service)

We have an on-going internal discussion to get this resolved but this is the current state of affairs, nothing has really progressed since we (core maintainers) first raised the access issue internally in late September.

Key immediate issues:

  • Auto-update bot is offline
  • Core maintainers do not have merge access in this repo (cdnjs/cdnjs)
  • Core maintainers do not have merge access (preferably admin access) to other repos in org
    • Such as cdnjs/new-website

Key long-term issues:

  • Current repo structure is not maintainable
    • Repo is extremely large and a pain to work with locally
    • Whilst the CI on PRs is good, it requires human intervention still
  • Auto-update/CI bot lacks some important functionality
  • No direct financial sponsorship (or any funding) for core maintainers to work on cdnjs

Update 1:

I reached out again to the founder in question yesterday (October 20th) to see if we can get any progress as we have both Microsoft (in #13500) and Greensock [GSAP] (in my Twitter DMs), who are both "big players", asking what's going on.

I actually got a response that they're going to sort something so that we (core maintainers) at least have access to merge PRs in this repo, nothing more though so far...

Update 2:

We (core maintainers) now (October 22nd) have access to merge PRs on cdnjs/cdnjs but the auto-update bot is still offline (as was originally described in this issue), so merging new and updated package.json PRs will achieve nothing as there is no bot to detect them and pull in new assets. We are still working internally to resolve this.

Update 3:

I followed up internally again on the 29th October regarding the two current key issues (auto-update & repo access). This issue has then made it to the front page of HackerNews (1st November) and still there has not been any progress internally, just radio silence. I'm really sorry we haven't got this sorted yet. :(

Update 4:

One of our founders has reappeared and granted Cloudflare full owner access to the cdnjs org to get things fixed. I believe that Cloudflare are going to have one of their engineers work to get things back up and running again. You can see the full comment from Zack below: #13524 (comment)

@drewfreyling drewfreyling mentioned this issue Oct 22, 2019
12 of 16 tasks complete
@MattIPv4 MattIPv4 changed the title no commit recently? Lack of cdnjs activity Oct 22, 2019
@MattIPv4 MattIPv4 pinned this issue Oct 22, 2019
@olafcm

This comment has been hidden.

@MattIPv4

This comment has been hidden.

@gruselhaus

This comment has been hidden.

@MattIPv4

This comment has been hidden.

kevin-brown added a commit to select2/select2 that referenced this issue Oct 31, 2019
This is needed until cdnjs figures out their future in
cdnjs/cdnjs#13524. Until then, we will just reference their existence
and promote jsDelivr to the primary source.
@jimaek

This comment has been hidden.

@MattIPv4

This comment has been hidden.

@jimaek

This comment has been hidden.

@aditsachde

This comment has been hidden.

@MattIPv4

This comment has been hidden.

@zackbloom
Copy link
Collaborator

@zackbloom zackbloom commented Nov 1, 2019

I work at Cloudflare on the team which supports CDNJS. To be very clear, any site using CDNJS will continue to get the files it relies on without interruption. We're committed to keeping CDNJS updated, performant, and reliable, for as long as the Internet chooses to use it.

We have engineers currently working with the CDNJS team to get the updating happening again. Once that is completed we can start thinking about the best long-term model to ensure projects stay updated without another incident like this. We deeply appreciate the patience and help from the CDNJS team, the users of CDNJS, and of course @MattIPv4, as this gets sorted.

@drewfreyling
Copy link
Member

@drewfreyling drewfreyling commented Nov 1, 2019

Thanks @zackbloom and @MattIPv4 - let me know if I can be of assistance with any of the development effort.

@PeterDaveHello
Copy link
Contributor

@PeterDaveHello PeterDaveHello commented Nov 1, 2019

Sorry for the inconvenience caused here, never mean to harm the project or Cloudflare.

It's really a ... hard decision to make to step down and take a rest, on a project I spent so much time and varies of resources on it, actually, I'm a little bit tired, too many things need to be done, resources are really limited, only few contributors get involved into it, instead, so many people take it as granted ...

Anyway, yes, we're working with Cloudflare right now to see how we can get it sorted, I hope it won't take too long.

@zackbloom
Copy link
Collaborator

@zackbloom zackbloom commented Nov 4, 2019

Thank you @PeterDaveHello. We all deeply appreciate all the time and effort you have put into CDNJS over the past few years, it is an amazing thing to go back through the commits and see just how much you have done.

@zackbloom zackbloom closed this Nov 20, 2019
@MattIPv4 MattIPv4 unpinned this issue Nov 20, 2019
alrra added a commit to alrra/browser-logos that referenced this issue Dec 1, 2019
Remove the `cdnjs` recommendation as the hosted copy of this project
is no longer kept up to date¹, and their update process seems to not
be fully automated².

- - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - -

¹ #181
² cdnjs/cdnjs#13606 (comment)

See also:

  * #181
  * cdnjs/cdnjs#13524

Fix #181
@mrlife
Copy link

@mrlife mrlife commented Jan 22, 2020

Thank you for providing CDNJS. Any update on the auto-update bot coming online? Not sure whether to keep waiting or shift to another CDN solution.

DataTables/DataTablesSrc#153

@MattIPv4
Copy link
Member

@MattIPv4 MattIPv4 commented Jan 22, 2020

The old auto-update bot is online and running. You can see that in the commit activity on the repo. We are still working on figuring out the best plan for a rewrite of the auto-update bot.

@mrlife
Copy link

@mrlife mrlife commented Jan 22, 2020

@MattIPv4 Thanks for letting me know. When will tagged releases start being picked up, e.g. DataTables 1.10.20?

@MattIPv4
Copy link
Member

@MattIPv4 MattIPv4 commented Jan 22, 2020

Looks like that was published quite a while back - I would guess that the file structure has changed or another issue has occurred with the auto-update of it that means manual intervention will be needed. Please create a separate issue requesting the library is updated here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
10 participants
You can’t perform that action at this time.