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

ENTERPRISE-61: Add Jenkins integration build faild documentation #113

Merged
merged 5 commits into from
Apr 20, 2017

Conversation

vinokurig
Copy link

Add Jenkins integration build faild documentation

codenvy-docs.iml Outdated
@@ -0,0 +1,22 @@
<?xml version="1.0" encoding="UTF-8"?>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why is this file needed? What does it do?

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oops. IntelliJ file. Needs to be removed.

Copy link
Contributor

@bmicklea bmicklea left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@vinokurig and @eivantsov I committed some tweaks to the docs page to make it more readable. Please ensure that all documentation is still accurate.

@@ -7,41 +7,56 @@ permalink: /:categories/continuous-integration/
---
{% include base.html %}

Codenvy is connected to your repo so any change made to the repo that would normally trigger a CI job will continue to trigger a CI job when the change is made in Codenvy.
Codenvy can use [Factories]({{base}}{{site.links["factory-getting-started"]}}) with a Jenkins CI system to generate developer workspaces pre-configured with the context of the CI job. For example, a failed CI build email can be customized to include a link to two factories that open a workspace pre-configured to:
* The branch and commit ID that broke the build for the project.
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

branch is not present in failed build factory. Project in factory will be checked out to commit.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The branch isn't specified in the Factory, but if the jenkins build is for a specific branch then the factory will use that branch, right?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think I have corrected this now, please review.

Codenvy is connected to your repo so any change made to the repo that would normally trigger a CI job will continue to trigger a CI job when the change is made in Codenvy.
Codenvy can use [Factories]({{base}}{{site.links["factory-getting-started"]}}) with a Jenkins CI system to generate developer workspaces pre-configured with the context of the CI job. For example, a failed CI build email can be customized to include a link to two factories that open a workspace pre-configured to:
* The branch and commit ID that broke the build for the project.
* The head of the master branch for the project.
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That link doesn't clarify things for me. If the jenkins job is setup to build master then the head factory will open with a clone of master, won't it? If the jenkins job is setup to build "brad" branch then the factory will open with a clone of "brad" right?

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

HEAD of branch. master is a branch too. So, if it is master then it is head of master.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fixed


#### For GitHub

Update the `codenvy.env` with `GitHub` webhooks properties. Note that you can rename "WEBHOOK1" with any identifier. Webhooks are tied to connectors through Factory IDs. There can be multiple webhooks (if you need to receive webhooks from multiple repositories) - in this case, add a new entry with a different name for WEBHOOKID and FACTORY1_ID.
Copy link
Author

@vinokurig vinokurig Apr 20, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

in this case, add a new entry with a different name for WEBHOOKID and FACTORY1_ID.

I would remove FACTORY1_ID from this description, it is described below

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok


```text
CODENVY_GITHUB_WEBHOOK_WEBHOOK1_REPOSITORY_URL=https://github.com/example/testrepo.git
CODENVY_GITHUB_WEBHOOK_WEBHOOK1_FACTORY1_ID=factory1Id
```

One webhook can update more than one Factory which can be added to configuration with a different Factory identifier, for example `FACTORY2_ID`:
One webhook can update more than one Factory. Add a different Factory identifier, for example `FACTORY2_ID`:
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would rephrase this:
Each webhook can update more than one Factory - in this case, add a new entry with a FACTORY_ID, where N is incremented number of the factory in the current group of entries, for example ...

the same for Bitbucket Server webhooks

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Okay - I also noticed that the BitBucket Server section was using a github URL so I replaced that.


#### BitBucket Server

Update the `codenvy.env` with `Bitbucket Server` webhooks properties. Webhooks are tied to connectors through Factory IDs. There can be multiple webhooks (if you need to receive webhooks from multiple repositories) - in this case, add a new entry with a different name for WEBHOOKID and FACTORY1_ID.
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

FACTORY1_ID is not needed here, see comment above

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok

@vinokurig
Copy link
Author

@bmicklea please review my comments

Brad Micklea added 2 commits April 20, 2017 11:29
@bmicklea bmicklea added this to the 5.9.0 milestone Apr 20, 2017
@bmicklea bmicklea merged commit 4c02141 into master Apr 20, 2017
@bmicklea bmicklea deleted the ENTERPRISE-61 branch April 20, 2017 15:53
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.

2 participants