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

[DNM] rpm: drop support for unsupported distros #6225

Closed
wants to merge 7,377 commits into from

Conversation

Projects
None yet
@smithfarm
Copy link
Contributor

commented Oct 11, 2015

@smithfarm smithfarm added this to the infernalis milestone Oct 11, 2015

@smithfarm smithfarm changed the title ceph.spec.in: drop support for RHEL<7 and SUSE<1210 in infernalis and above [DNM] ceph.spec.in: drop support for RHEL<7 and SUSE<1210 in infernalis and above Oct 11, 2015

@smithfarm

This comment has been minimized.

Copy link
Contributor Author

commented Oct 11, 2015

Note: this breaks ceph-common for older distros.

@smithfarm

This comment has been minimized.

Copy link
Contributor Author

commented Oct 12, 2015

Passes gitbuilder.

@liewegas

This comment has been minimized.

Copy link
Member

commented Oct 12, 2015

We can do this post-infernalis.

Also, although we aren't building ceph.com packages on old distros, in theory you could if you go and install the right compiler toolchain yourself. Removing this cruft is nice but it means any backporter will also have to futz around with the spec file too... we may want to wait a bit longer?

@ktdreyer

This comment has been minimized.

Copy link
Member

commented Oct 12, 2015

I'm ok with merging this before infernalis, or after. Either way works for me

@smithfarm

This comment has been minimized.

Copy link
Contributor Author

commented Oct 12, 2015

@liewegas has a point. Though upstream will not be testing infernalis against older distros, somebody out there might be interested in doing so on their own, and this PR would make life difficult for them.

So, the question becomes: retarget against master and merge now, or close the PR and reopen it post-jewel?

@liewegas

This comment has been minimized.

Copy link
Member

commented Oct 14, 2015

As much as I would love to remove this cruft, I think we should carry it for a bit longer.

Let's see how the no-el6 support pans out for infernalis. If nobody cares, we could merge this before jewel. If people end up doing these builds downstream we can carry this stuff a bit longer.

@liewegas liewegas closed this Oct 14, 2015

dillaman and others added some commits Apr 13, 2016

journal: race possible when re-watching a journal object
Signed-off-by: Jason Dillaman <dillaman@redhat.com>
rgw: RGWPeriod::reflect() sets master zonegroup as default
if RGWPeriod::reflect() is called right after a 'realm pull', it will
create the zonegroup objects but not set any as default.  so the next
time RGWRados is initialized, create_defaults() will add a new 'default'
zonegroup and set it as the master.  this causes a 'period update' to
find multiple master zonegroups and fail with EINVAL

reflect() now calls set_as_default(exclusive=true) on the master
zonegroup to guarantee that a default is set

Signed-off-by: Casey Bodley <cbodley@redhat.com>
Merge pull request #8498 from liewegas/wip-bluestore
os/bluestore: fix shortened bluefs paths in env mirror case
test: image replayer needs dummy remote mirror peer uuid
Signed-off-by: Jason Dillaman <dillaman@redhat.com>

@smithfarm smithfarm modified the milestones: kraken, infernalis Apr 13, 2016

@smithfarm smithfarm reopened this Apr 13, 2016

@smithfarm

This comment has been minimized.

Copy link
Contributor Author

commented Apr 13, 2016

@ktdreyer This needs rebase, but I think it does some of what you are asking for in http://tracker.ceph.com/issues/15486

cbodley and others added some commits Apr 13, 2016

rgw: call rgw_log_usage_finalize() on reconfiguration
RGWRealmReloader was calling rgw_log_usage_init() with the new RGWRados
instance, but never cleaned up the previous one with
rgw_log_usage_finalize()

Signed-off-by: Casey Bodley <cbodley@redhat.com>
systemd: Use the same restart limits as upstart
Currently, the systemd daemons are not restarted on failure. This patch
adds this functionality and sets the defaults to those defined in
upstart. This resolves to 3 fails per 30 minutes for osd, mon and mds
and 5 fails per 30 seconds for radosgw.

Signed-off-by: Boris Ranto <branto@redhat.com>
doc: fix dependencies
Signed-off-by: Etienne Menguy <etienne.menguy@corp.ovh.com>
osd/Watch: slightly more informative debug output
Signed-off-by: Sage Weil <sage@redhat.com>
osd: fix watch reconnect race
It's possible to process a watch reconnect op, setting Watch::conn, before
the ms_handle_reset path tries to disconnect the old one.  Since reset()
was blindly disconnecting the "current" connection, we could race and
disconnect the new con instead of the old one.

Fix this by specifying which con to disconnect.

Fixes: http://tracker.ceph.com/issues/15441
Signed-off-by: Sage Weil <sage@redhat.com>
@ktdreyer

This comment has been minimized.

Copy link
Member

commented Apr 13, 2016

Great point @smithfarm , this would make a good starting point.

liewegas and others added some commits Apr 13, 2016

Merge pull request #8188 from ceph/wip-systemd-restart-defaults
systemd: Use the same restart limits as upstart

Reviewed-by: Ken Dreyer <kdreyer@redhat.com>
Reviewed-by: Greg Farnum <gfarnum@redhat.com>
rgw: try to use current period id in a few more cases
If missing period id, try to use the current period id. This is needed
for sync agent compatibility.

Signed-off-by: Yehuda Sadeh <yehuda@redhat.com>
Merge pull request #8584 from dillaman/wip-15479
test: image replayer needs dummy remote mirror peer uuid

Reviewed-by: Josh Durgin <jdurgin@redhat.com>

majianpeng and others added some commits Apr 22, 2016

os/bluestore/BlueFS: Record the correctly perfcouter.
After "log_writer->append(bl)", the length of bl become zero.
So tune the order.

Signed-off-by: Jianpeng Ma <jianpeng.ma@intel.com>
Merge pull request #8551 from scienceluo/patch-1
doc: Fixes Different font of the title

Reviewed-by: Abhishek Lekshmanan <abhishek@suse.com>
Reviewed-by: Yatin Kumbhare <yatinkumbhare@gmail.com>
Merge pull request #8549 from tchaikov/wip-kill-filestore-warning
pybind, FileStore: silence gcc warnings

Reviewed-by: Haomai Wang <haomai@xsky.com>
Merge pull request #8527 from roidayan/wip-vstart_clean_prints
vstart: clean prints

Reviewed-by: Sage Weil <sage@redhat.com>
rgw: fix memory leak in func 'RGWCreateBucket_ObjStore_S3::get_params' (
#8670)

Signed-off-by: weiqiaomiao <wei.qiaomiao@zte.com.cn>
Reviewed-by: Casey Bodley <cbodley@redhat.com>
doc/release-notes: revise terminology
Avoid loaded 'production ready' term in favor of just 'stable'.

Signed-off-by: Sage Weil <sage@redhat.com>
Merge pull request #8705 from liewegas/wip-jewel
doc/release-notes: revise terminology

Reviewed-by: Greg Farnum <gfarnum@redhat.com>
doc/release-notes: is, not will be, basis for LTS
Signed-off-by: Sage Weil <sage@redhat.com>
Merge pull request #8706 from liewegas/wip-jewel
doc/release-notes: is, not will be, basis for LTS

Reviewed-by: Abhishek Lekshmanan <abhishek@suse.com>
Merge pull request #8463 from majianpeng/bluestore-zero-truncate
os/bluestore/BlueStore: Don't forget assign nid when offset > size in _do_zero

Reviewed-by: Sage Weil <sage@redhat.com>
Merge pull request #8458 from jjhuo/clean_blue_configs
common/config_opts: remove invalid config options of bluestore

Reviewed-by: Kefu Chai <kchai@redhat.com>
Merge pull request #8449 from javacruft/ceph-osd-prestart
ceph-osd-prestart.sh: drop --setuser/--setgroup

Reviewed-by: Sage Weil <sage@redhat.com>
Merge pull request #8395 from renhwsky/renhw-wip-monmap
mon/monmap:  conditional optimization in build_from_host_list

Reviewed-by: xie xingguo <xie.xingguo@zte.com.cn>
Tamil Muthamizhan
Signed-off-by: Tamil Muthamizhan <tmuthami@redhat.com>
  renamed the workunit and test to reflect reality and for better clarity
Merge pull request #8710 from ceph/wip-rename-tests
Signed-off-by: Tamil Muthamizhan <tmuthami@redhat.com>
Merge pull request #8596 from xiexingguo/xxg-wip-blue
os/filestore: add sanity check for stat() syscall

Reviewed-by: Sage Weil <sage@redhat.com>
Merge pull request #8577 from runsisi/wip-fix-op-lock
osdc/Objecter: small cleanup for osd op completion lock

Reviewed-by: Sage Weil <sage@redhat.com>
Merge pull request #8591 from xiexingguo/xxg-wip-15492
test: generate temp pool name in a safer way

Reviewed-by: Sage Weil <sage@redhat.com>
Merge pull request #8384 from songbaisen/song18
crush: when take place the crush map should consider the rule is in …

Reviewed-by: Sage Weil <sage@redhat.com>
Merge pull request #7997 from songbaisen/song11
osdc: Reverse the probing vector in better way.

Reviewed-by: Kefu Chai <kchai@redhat.com>
Merge pull request #7942 from xiexingguo/xxg-wip-fixdirtyinc
mon: OSDMonitor: improve pool creation logic

Reviewed-by: Kefu Chai <kchai@redhat.com>
Reviewed-by: Xiaoxi Chen <xiaoxchen@ebay.com>

@smithfarm smithfarm changed the title [DNM] ceph.spec.in: drop support for RHEL<7 and SUSE<1210 in infernalis and above [DNM] rpm: drop support for unsupported distros Apr 23, 2016

smithfarm added some commits Oct 11, 2015

ceph.spec.in: drop python_sitelib/sitearch conditional
This conditional was required to support older versions of RHEL/CentOS that are
no longer supported in infernalis and above.

Signed-off-by: Nathan Cutler <ncutler@suse.com>
rpm: drop init-ceph.in-fedora.patch
Signed-off-by: Nathan Cutler <ncutler@suse.com>
ceph.spec.in: drop _with_systemd
Ceph versions jewel and above only support systemd.

Signed-off-by: Nathan Cutler <ncutler@suse.com>

@smithfarm smithfarm force-pushed the SUSE:wip-13445-infernalis branch from 55d2992 to 18afa3e Apr 23, 2016

ceph.spec.in: drop python-argparse dependency
This was only necessary for older (now unsupported) distro versions.

Signed-off-by: Nathan Cutler <ncutler@suse.com>

@smithfarm smithfarm closed this Apr 23, 2016

@smithfarm smithfarm deleted the SUSE:wip-13445-infernalis branch Apr 23, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.