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

#185 Scheduled posting of CloudWatch data #188

Merged
merged 4 commits into from
Jul 10, 2014
Merged

Conversation

carlosmiranda
Copy link
Collaborator

No description provided.

@davvd
Copy link

davvd commented Jul 4, 2014

I'll find a reviewer for your pull requests shortly, thanks for contribution!

@davvd
Copy link

davvd commented Jul 4, 2014

@yegor256 please review

@yegor256
Copy link
Owner

yegor256 commented Jul 8, 2014

@rultor good to merge

@rultor
Copy link
Collaborator

rultor commented Jul 8, 2014

@rultor good to merge

@yegor256 OK, I'm on it. You can track me here

@rultor
Copy link
Collaborator

rultor commented Jul 8, 2014

Oops, I failed to merge. Full log is here

@yegor256
Copy link
Owner

yegor256 commented Jul 8, 2014

@carlosmiranda I'm testing new version of Rultor, so please be ready for a few false runs

@yegor256
Copy link
Owner

yegor256 commented Jul 8, 2014

@rultor should be fine now, good to merge

@rultor
Copy link
Collaborator

rultor commented Jul 8, 2014

@rultor should be fine now, good to merge

@yegor256 OK, I'm on it. You can track me here

@rultor
Copy link
Collaborator

rultor commented Jul 8, 2014

Oops, I failed to merge. Full log is here

@yegor256
Copy link
Owner

yegor256 commented Jul 9, 2014

@rultor again, good to merge

@rultor
Copy link
Collaborator

rultor commented Jul 9, 2014

@rultor again, good to merge

@yegor256 OK, I'm on it. You can track me here

@rultor
Copy link
Collaborator

rultor commented Jul 9, 2014

Oops, I failed to merge. Full log is here

@yegor256
Copy link
Owner

yegor256 commented Jul 9, 2014

@carlosmiranda I think it's a valid error reported by rultor above. When I run it locally, I get the same problem. What do you think?

@carlosmiranda
Copy link
Collaborator Author

@yegor256 Yes, it looks like a valid error. Some of our previous Travis builds also failed with similar messages (although they've been failing with OutOfMemoryError/PermGen space messages recently). For example, see. https://travis-ci.org/yegor256/s3auth/builds/27464875 . They may be related.

Please give me some time to investigate. Meanwhile I think we can try merging with our other projects?

@yegor256
Copy link
Owner

yegor256 commented Jul 9, 2014

yes, I'll try new rultor with other projects right now

@carlosmiranda
Copy link
Collaborator Author

@yegor256 , let's try again. The new jcabi-manifests should fix it.

@yegor256
Copy link
Owner

@rultor good to merge

@rultor
Copy link
Collaborator

rultor commented Jul 10, 2014

@rultor good to merge

@yegor256 OK, I'm on it. You can track me here

@rultor rultor merged commit e79a0fe into yegor256:master Jul 10, 2014
@yegor256
Copy link
Owner

@carlosmiranda you were right, it was a problem in jcabi-manifests :)

@rultor
Copy link
Collaborator

rultor commented Jul 10, 2014

Done! FYI, full log is here

@carlosmiranda
Copy link
Collaborator Author

@yegor256 , indeed it was. :) Unfortunately the Travis build is still broken and I still don't know what's causing it... but we have a separate ticket for that already

@davvd
Copy link

davvd commented Jul 14, 2014

@yegor256 I just added 36 mins to your account, many thanks for your contribution..

@rultor
Copy link
Collaborator

rultor commented Jul 14, 2014

@rultor good to merge

@yegor256 pull request is closed already, I can't merge it

@rultor
Copy link
Collaborator

rultor commented Jul 14, 2014

@rultor should be fine now, good to merge

@yegor256 pull request is closed already, I can't merge it

@rultor
Copy link
Collaborator

rultor commented Jul 14, 2014

@rultor again, good to merge

@yegor256 pull request is closed already, I can't merge it

@rultor
Copy link
Collaborator

rultor commented Jul 14, 2014

@rultor good to merge

@yegor256 pull request is closed already, I can't merge it

@davvd
Copy link

davvd commented Jul 15, 2014

@rultor please deploy

@carlosmiranda carlosmiranda deleted the 185 branch July 19, 2014 03:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants