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

Switch node host providers #425

Closed
Martii opened this issue Nov 18, 2014 · 9 comments
Closed

Switch node host providers #425

Martii opened this issue Nov 18, 2014 · 9 comments
Labels
CODE Some other Code related issue and it should clearly describe what it is affecting in a comment. migration Use this to indicate that it may apply to an existing or announced migration.

Comments

@Martii
Copy link
Member

Martii commented Nov 18, 2014

Through all of the deployment woes that we've been having for well over a week it's probably about time to hear some alternatives for hosting OUJS somewhere more reliable.

Basically the recent issue is with npm and file permissions with remote npm not being configured properly on their end.

As I have constantly stated certain items would be best served within our realm of control... unfortunately since nodejitsu/jitsu appears to not honor the r switch and is very buggy just in general when deployed from multiple locations it is about time to consider the alternatives.

Any ideas or suggestions?

@Martii Martii added the team biz This is similar to a meta discussion. label Nov 18, 2014
@Zren
Copy link
Contributor

Zren commented Nov 18, 2014

What deployment issues?

@Martii
Copy link
Member Author

Martii commented Nov 18, 2014

We weren't able to deploy until today... after 8ish days of talking back and forth with nodejitsu. Sizzle temporarily handed over deployment to me yesterday ... which has been relinquished... and I discovered that jitsu is flawed (the deployment app nodejitsu uses on their dummy front-end)... and the backend scripts which accept the tokens and basic commands and then calls whatever scripts that are closed source on their end.

Basically my last correspondence with nodejitsu was that the r switch isn't supported and we shouldn't even think about trying to use it... even though it's in the options. One of my issues mentioned using the gh hash which can easily be achieved with the follow deploy script:

#!/bin/bash
HEAD=`git rev-parse --short HEAD`
VER=`node -pe 'JSON.parse(process.argv[1]).version' "$(cat package.json)"`

jitsu deploy -c -r $VER-hash-$HEAD

unfortunately this causes nodejitsu to bomb miserably.... even with a custom numerical commit instead of a sha1 sum clipped hash it now fails. (numerical was working about 5 hours ago until they downgraded npm for the initial reason why we couldn't deploy).

Basically nodejitsu in my humble opinion is unreliable at this time... too many problems that are completely out of our control. Perhaps a virtual Linux host would be better suited for this.

@Martii
Copy link
Member Author

Martii commented Nov 18, 2014

You may also notice that our current version on production is foo'd since this is a different location than sizzle... see https://openuserjs.org/about#welcome with the lack of -104... jitsu is definitely bad about this across systems... @sizzlemctwizzle has been made aware of all of this too... which is why we should have the gh commit hash... also our snapshots are foo'd in the history on nodejitsu and their service says that we have multiple snapshots active. (we already have a snapshot of 0.1.3 when that was bumped and now we are back to it... so duplication).

@Zren
Copy link
Contributor

Zren commented Nov 18, 2014

What does the r switch do? Why do we need to use it?

@Martii
Copy link
Member Author

Martii commented Nov 18, 2014

Sets the ./package.json version for deployment. npm will fail if it is not semver x.x.x but that is why the -whatever exists. You can see how it is being successfully used on npmjs.org with https://www.npmjs.org/package/select2 ... anyhow... because I deployed... the value that it increments is pretty much random... sizzles "jitsu deploy version head" is at "-103" right now ... so his next should be "-104"... but that's not always the case when deployment happens across multiple systems... at one point the "auto deploy version" skipped to "-10" and then silently did "-104" and then did it again at "-104" then we are at "-106"... basically this is the auto-numbering flaw... but the fact is if we had the GH commit hash this wouldn't mess up our snapshots... e.g. we could go back to a specific version ... but because we have two three of "0.1.3" now... it is impossible to revert since all are considered active in their backend.

Anyhow... basically nodejitsu isn't reliable at this point in time and imo historically since I've been active with this project. I've done my best to work with their support on this and their last reply was to use their auto-naming system instead of a feature they claim to support... it's been a total headache for sizzle and a minor one for me trying to debug their application... but since jitsu is just a dummy front for their backend scripts most of the issues were remote... and now they've killed/broken the r switch altogether in their backend.

@Martii
Copy link
Member Author

Martii commented Nov 18, 2014

If anyone is still wondering if you change https://github.com/OpenUserJs/OpenUserJS.org/blob/master/package.json#L4 to be "0.1.3-hash-9c43c59" (that's our current head and normalized so it could be hyperlinked to https://github.com/OpenUserJs/OpenUserJS.org/tree/9c43c59) and run dev... npm and node currently have zero issues with this... also the "-browserify-fix" on select2 has no issues with the npmjs registry... so nodejitsu/jitsu still has a validation issue in their backend.

@Martii
Copy link
Member Author

Martii commented Nov 25, 2014

Arrgh!

...
info:    Creating snapshot 0.1.4-4
info:    Uploading: [=============================] 100%
error:   Error running command deploy
error:   Error building snapshot
error:   Nodejitsu Error (500): Internal Server Error
...
info:    Nodejitsu not ok

... this isn't helpful. :(


Snapshot was built... uploaded okay... then it claims the snapshot couldn't be built...WHAT!?!?! ... this is a paradox in nomenclature and very non-descriptive.


See also:

@Martii
Copy link
Member Author

Martii commented Feb 11, 2015

... Nodejitsu products will continue to run for the next seven months, then shutdown. ...

... from http://blog.nodejitsu.com/nodejitsu-joins-godaddy/

Cc: @sizzlemctwizzle

@Martii Martii changed the title Consider switching node host providers Switch node host providers Feb 11, 2015
@Martii Martii added sooner Sooner would be appreciated. and removed team biz This is similar to a meta discussion. labels Feb 11, 2015
@Martii Martii added expedite Immediate and on the front burner. and removed sooner Sooner would be appreciated. labels Feb 25, 2015
@Martii Martii added the tracking upstream Waiting, watching, wanting. label Mar 27, 2015
@Martii Martii mentioned this issue Mar 27, 2015
@Martii Martii added this to the #425 milestone Apr 27, 2015
Martii pushed a commit to Martii/OpenUserJS.org that referenced this issue May 1, 2015
* Removes the previous exception found with *fakeS3* and all importing/creation of scripts.
* Bump `./package.json` to reflect tested nodejs major versions... **do not use 0.12.0 on nodejitsu due to previous deploy failures during 0.10.33 static testing

**NOTE** Regression tested on 0.10.xx and appears okay... retested on local pro. There are some deprecation warnings that may not be able to be addressed until node host is upgraded in OpenUserJS#425

Applies to OpenUserJS#581
@Martii Martii added migration Use this to indicate that it may apply to an existing or announced migration. CODE Some other Code related issue and it should clearly describe what it is affecting in a comment. and removed expedite Immediate and on the front burner. tracking upstream Waiting, watching, wanting. labels May 30, 2015
@Martii
Copy link
Member Author

Martii commented May 30, 2015

Closed by #635

@Martii Martii closed this as completed May 30, 2015
Martii pushed a commit to Martii/OpenUserJS.org that referenced this issue Jul 8, 2015
* Since we've switched to a VPS this test is no longer needed and will help keep production up to date.

Post OpenUserJS#425 fix
@OpenUserJS OpenUserJS locked as resolved and limited conversation to collaborators Apr 12, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
CODE Some other Code related issue and it should clearly describe what it is affecting in a comment. migration Use this to indicate that it may apply to an existing or announced migration.
Development

No branches or pull requests

3 participants