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

ceph: Enable separate builds of ceph & rgw FSALs #16

Open
wants to merge 1 commit into
base: centos-ci
Choose a base branch
from

Conversation

alimaredia
Copy link

  • remove ceph packages/testing in fsal gluster
    builds
  • Ceph packages are the lastest builds of master
    from shaman.ceph.com

Signed-off-by: Ali Maredia amaredia@redhat.com

- remove ceph packages/testing in fsal gluster
builds
- Ceph packages are the lastest builds of master
from shaman.ceph.com

Signed-off-by: Ali Maredia <amaredia@redhat.com>
@alimaredia
Copy link
Author

alimaredia commented Sep 15, 2017

@nixpanic @thotz, @ffilz told me that I need to enable logging somehow? Any pointers on how to do that?

@ffilz
Copy link
Member

ffilz commented Sep 15, 2017

What I was getting at with "logging" was how does someone know these tests failed on their patch.

I think this would ideally run with a different gerrithub ID so that it can have a separate Verified "vote" on patches.

I don't understand how all that magic works though...

@dang
Copy link

dang commented Sep 18, 2017

@ffilz This currently has no hooks into the CI yet. This should go in as a new ceph branch, and then we can start on the CI hooks (since the CI needs some place to check out from before it can run). This will have a different ID.

@alimaredia alimaredia changed the title ceph: Enable seperate builds of ceph & rgw FSALs ceph: Enable separate builds of ceph & rgw FSALs Sep 22, 2017
Copy link

@nixpanic nixpanic left a comment

Choose a reason for hiding this comment

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

The majority is probably a copy/paste from an other job, which one? There might be similar mistakes in there.

Testing Ceph would be better when there is a Ceph bot account for gerrithub.io. This way we (and Frank!) can see which projects have voted. Reqesting a bot account is relatively easy, send an email to info@gerritforge.com (give them a public ssh key) and we can work on adding the bot to the CentOS CI Jenkins instance (through bstinson in #centos-devel on Freenode).

RET=$?

# exit with SUCCESS or FAIL only
exit ${EXIT}</command>
Copy link

Choose a reason for hiding this comment

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

Where does ${EXIT} come from?

# only use https for now
GIT_REPO="https://${GERRIT_HOST}/${GERRIT_PROJECT}"

SHAMAN_REPO_URL="https://shaman.ceph.com/api/repos/ceph/master/latest/centos/7/flavors/default/repo"
Copy link

Choose a reason for hiding this comment

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

You should use ${CENTOS_VERSION} instead of 7. CentOS-6 can be provided through the jobs too (and that should probably fail while installing dependencies).

<choices class="java.util.Arrays$ArrayList">
<a class="string-array">
<string>7</string>
<string>6</string>
Copy link

Choose a reason for hiding this comment

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

Are you sure you want to provide testing for CentOS-6?

<choices class="java.util.Arrays$ArrayList">
<a class="string-array">
<string>x86_64</string>
<string>i386</string>
Copy link

Choose a reason for hiding this comment

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

Are there i386 compatible RPMs in the shaman repository?

@thotz thotz removed their assignment Jun 10, 2021
@rakshithakamath94 rakshithakamath94 force-pushed the centos-ci branch 19 times, most recently from d8d767f to cf571a6 Compare June 19, 2023 15:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants