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

Failed Publish Operations #11

Open
splendido opened this issue Mar 1, 2015 · 22 comments
Open

Failed Publish Operations #11

splendido opened this issue Mar 1, 2015 · 22 comments

Comments

@splendido
Copy link
Member

Failed publish requests received on autopublish.meteor.com will be reported here.

Please investigate and report back...

After a publish operation is solved, please delete all related messages to keep this track clear!

@dandv
Copy link
Contributor

dandv commented Mar 2, 2015

All failures in this same issue?

Would it be possible to @-mention the user?

@splendido
Copy link
Member Author

Yep, the mention is possible, and useful for sure...
What I get for free is the user who pushed the new release, but might not be the user to warn if we keep separate wrapper repos...

All notification under the same issue might reduce the pain to get 10-20 new issues on failure bursts (see the case for semantic-ui...)

@zimme
Copy link
Member

zimme commented Apr 2, 2015

I'm thinking maybe pushing emails to the maintainers of the packages is better then this thread?

@splendido
Copy link
Member Author

@zimme, not all maintainers would be willing to receive annoying emails (see official integrations of 3rd party libraries for which we asked the upstream maintainers to register their own meteor user...)

But I agree that we should have some more advanced configuration options to let maintainers choose about this.

I thought about this thread because I supposed most of the errors could have been due to problems into the autopublish code, so basically I wanted to be warned when something went wrong...
And this prooved to be effective for the past meteor updates for which I had to manually update the meteor version on my build machine ;-)

What I could add for free (if I found the time...) is a mention to the author on this very same thread (as @dandv already suggested...)

@tredondo
Copy link

Publish for package packaging:autopublish-test failed, please revise!

@tredondo
Copy link

Publish for package zimme:iron-router-active failed, please revise!

@zimme
Copy link
Member

zimme commented May 19, 2015

@splendido, the publication actually went through, zimme:[email protected] got published fine.

@splendido
Copy link
Member Author

The log seems interrupted...
I'd say there' some problem with the code used to get the ssh connection and run commands :(
I'll have to look it better: I already had the feeling it crashed other times...

@splendido
Copy link
Member Author

but wasn't it zimme:iron-router-active?

@splendido
Copy link
Member Author

actually the log seems complete:

Publishing package build for zimme:ac...  |\r   Publishing package build for zimme:ac...  |\r   Publishing package build for zimme:ac...  |\rPublished zimme:[email protected].\r   Publishing package build for zimme:ac...  |\r   Updating package catalog                  |\r   Updating package catalog                  /\r   Updating package catalog                  /\r   Updating package catalog                  /\r   Updating package catalog                  /\r   Updating package catalog                  /\r   Updating package catalog                  -\r   Updating package catalog                  -\r   Updating package catalog                  -\r   Updating package catalog                  -\r   Updating package catalog                  -\r   Updating package catalog                  \\\r   Updating package catalog                  \\\r   Updating package catalog                  \\\r   Updating package catalog                  \\\r   Updating package catalog                  \\\r   Updating package catalog                  |\r   Updating package catalog                  |\r   Updating package catalog

and there's a Published zimme:[email protected] well before the end...
The thing is I'm using this regexp to verify successful publish and your package name is not aligned with the one on the subscription which is still zimme:iron-router-active

Unfortunately we're still not able to pick up a particular branch of a repository :(

@zimme
Copy link
Member

zimme commented May 19, 2015

Yeah, I'll rename the Github repo once 2.0.0 is stable then I won't see these errors anymore I guess.

But this is something to think about supporting. Mike with his mocha packages he publishes mocha-web from one branch and mocha-packages from another branch and if there's one user doing this, there might be others =)

@splendido
Copy link
Member Author

is a git tag branch agnostic?

@zimme
Copy link
Member

zimme commented May 19, 2015

a git tag tracks a specific commit, it's a bit like a branch. But it shouldn't care about which branch it was tagged in. a tag should just represent a specific commit in the code base

@splendido
Copy link
Member Author

so could/should we update the package name in the db reading it from the latest tag checked out before publish?
...or simply read the package name being published from package.js just before running meteor publish?
This could be a way to overcome the above error...

@splendido
Copy link
Member Author

...obviously won't fit the case for developers using different branches for different packages ;-)

@zimme
Copy link
Member

zimme commented May 19, 2015

Reading from package.js from the checked out code sounds like a plan, that way the name will always be correct, even if I work from a separate branch and create a tag from there, which I need to do for autopublish to "trigger"

@jlukic
Copy link

jlukic commented Jul 13, 2015

I just noticed autopublish is failing for our packages as of a few days ago
https://autopublish.meteor.com/log/tf22miRa42aRNqLhB

meteor --version
Meteor 1.1.0.2
meteor login
                                             /
                                             /
[1G[0JUsername: [11Gmeteorpublish
Password: 
                                             -
Login failed.
                                             -
meteor whoami
Not logged in. 'meteor login' to log in.

@jlukic
Copy link

jlukic commented Jul 22, 2015

@splendido Any idea what's the issue?

@splendido
Copy link
Member Author

@jlukic as you already noticed, the bot user is no more able to log in.
We've already understood what the problem is and we're in touch with MDG to solve the problem.

We'll let you know when everything will be back to normal.
I'm sorry for this.

@jlukic
Copy link

jlukic commented Jul 29, 2015

No prob. Just ping me when its back up

@splendido
Copy link
Member Author

I hope I'll have some answer in a couple of days...
Then we might think to change bot user and probably also the domain to simething like aurora.meteor.com, likely...

@splendido
Copy link
Member Author

Hey guys,
publishing operations are catching up all failed requests since 7/7.

we're now using a new bot user called publishbot which is the one you have to add to your organization or as a maintainer of your packages to be able to get them published via autopublish.meteor.com.

Please see the updated how it works page with the new instructions...

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

No branches or pull requests

5 participants