Comments (8)
@danmcp is @openshift-bot merging with master when we run tests in origin? my opinion is that you don't want to merge with master as you want to test the state of your tree. if we merge against master, the test might be inconsistent (it passes in jenkins, but fails in your branch and vice versa).
from vagrant-openshift.
It wouldn't be "Continuous Integration" if we don't actually integrate the code, no?
from vagrant-openshift.
@rhcarvalho ok, I think i'm fine merging it to master prior to test execution. what will happen when the merge fail?
from vagrant-openshift.
I believe the way this works is that the branch has already been rebased on top of master at this point.
So this is already happening. I think when the merge/rebase fails you get the "needs rebase" message and then it probably just tests your branch directly.
from vagrant-openshift.
@rhcarvalho @mfojtik https://github.com/openshift/test-pull-requests
from vagrant-openshift.
@bparees the job @rhcarvalho is referring to is managed by the Jenkins GH pull tester plugin...
from vagrant-openshift.
@rhcarvalho @mfojtik https://github.com/openshift/test-pull-requests
@bparees I believe the code I'm referring to is unrelated to that (I think I didn't even knew it existed, thanks for the link).
The Jenkins job, for example for MongoDB, is configured to run:
...
vagrant test-origin-image --image mongodb --source openshift/$JOB_NAME --ref $ghprbActualCommit
...
Then it goes as I pointed in the issue description, we clone the image repo, fetch all PR refs and checkout to the commit of the PR.
If a PR by itself works, but after integrating with the existing code (iow, after merge) the changes do not work, we're letting it slip.
from vagrant-openshift.
yeah i didn't recognize this was about the image tests, my bad.
you're right that the image tests just test the branch. Where the rubber hits the road for those is in the push images job which tests everything again before publishing an image. So we're still protected from pushing out a bad image, but it is not ideal to merge a broken PR and then leave the push job in a broken state until we revert the commit. That's never happened that i'm aware of, so this isn't a very high priority in my mind, but it's true that the current situation isn't ideal.
from vagrant-openshift.
Related Issues (20)
- `vagrant sync-origin` fails with stacktrace when no instance is up
- README should suggest `dnf install` for `vagrant-libvirt` HOT 2
- Permissions not correctly set-up for `--provider libvirt`
- Bash completion should be sourced by provisioning
- `vagrant sync-origin` fails when SSH override is busy
- Remove meta-reporting from commands run over SSH HOT 1
- vagrant origin-local-checkout does not support --branch HOT 3
- vagrant sync-origin fails with "cannot allocate memory" HOT 2
- libvirt provider hostname resolution not working HOT 1
- extended tests doesn't handle pipe correctly for focus HOT 1
- Cannot pass environment variables to extended tests HOT 1
- update docker config to do v1 pushes
- Stuck at Configuring and enabling network interfaces HOT 2
- Authentication failure HOT 5
- origin-local-checkout doesn't work with branches HOT 3
- vagrant sync-origin error HOT 2
- build in vagrant failed due to "cannot allocate memory"
- network unreachable HOT 33
- libvirt provider requires 50 GB of disk space HOT 3
- Should this repo really be archived? HOT 1
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from vagrant-openshift.