Skip to content
This repository was archived by the owner on Apr 12, 2024. It is now read-only.

ci.angularjs.org is having a bad week #9783

Closed
caitp opened this issue Oct 24, 2014 · 2 comments · Fixed by #9816
Closed

ci.angularjs.org is having a bad week #9783

caitp opened this issue Oct 24, 2014 · 2 comments · Fixed by #9816
Assignees
@jeffbcross
Milestone
1.3.1

Comments

@caitp
Copy link
Contributor

caitp commented Oct 24, 2014

@gkalpak informs me that code.angularjs.org's snapshot directory isn't being updated, I guess master jenkins jobs aren't running to completion. not sure why, travis is green.

@jeffbcross could you take a look at this when you have a chance, thanks =)
@jeffbcross
Copy link
Contributor

Looking....

@jeffbcross
Copy link
Contributor

I restarted the server because I saw some concerning real memory usage by some processes. Then when I ran master tests again, I noticed that all the E2E tests were 404ing "Cannot GET /build/docs/..."

The first job that started failing was when these commits were merged that affected e2e tests: 762713e
22b817e

I've got to head out, but will pick this back up tomorrow

@jeffbcross jeffbcross added this to the 1.3.1 milestone Oct 28, 2014
jeffbcross added a commit to jeffbcross/angular.js that referenced this issue Oct 28, 2014
@jeffbcross
fix(jenkins): reset baseUrl in protractor conf
c05edf3 
Commit 22b817e changed the url
used by protracts in all docs tests to add build/docs, which
was already set to the baseUrl in protractor-jenkins.conf. This
commit just changes the protractor config's baseUrl to adapt
to the changes in the spec files.

Closes angular#9783
@jeffbcross jeffbcross mentioned this issue Oct 28, 2014
Merged
jeffbcross added a commit to jeffbcross/angular.js that referenced this issue Oct 28, 2014
@jeffbcross
fix(jenkins): reset baseUrl in protractor conf
b4db713 
Commit 22b817e changed the url
used by protractor in all docs tests to prepend "build/docs", which
was already set to the `baseUrl` in `protractor-jenkins.conf`. This
commit just changes the protractor config's `baseUrl` to adapt
to the changes in the spec files.

Closes angular#9783
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@jeffbcross jeffbcross

Labels
None yet
Projects
None yet
Milestone
1.3.1
Development

Successfully merging a pull request may close this issue.

fix(jenkins): reset baseUrl in protractor conf jeffbcross/angular.js
2 participants
@jeffbcross @caitp