Show More
The requested changes are too big and content was truncated. Show full diff
|
1 | NO CONTENT: new file 100644 | |
The requested commit or file is too big and content was truncated. Show full diff |
@@ -0,0 +1,63 b'' | |||
|
1 | import logging | |
|
2 | ||
|
3 | from sqlalchemy import * | |
|
4 | from rhodecode.model import meta | |
|
5 | from rhodecode.lib.dbmigrate.versions import _reset_base, notify | |
|
6 | ||
|
7 | log = logging.getLogger(__name__) | |
|
8 | ||
|
9 | ||
|
10 | def get_by_key(cls, key): | |
|
11 | return cls.query().filter(cls.ui_key == key).scalar() | |
|
12 | ||
|
13 | ||
|
14 | def create_or_update_hook(cls, key, val, SESSION): | |
|
15 | new_ui = get_by_key(cls, key) or cls() | |
|
16 | new_ui.ui_section = 'hooks' | |
|
17 | new_ui.ui_active = True | |
|
18 | new_ui.ui_key = key | |
|
19 | new_ui.ui_value = val | |
|
20 | ||
|
21 | SESSION().add(new_ui) | |
|
22 | ||
|
23 | ||
|
24 | def upgrade(migrate_engine): | |
|
25 | """ | |
|
26 | Upgrade operations go here. | |
|
27 | Don't create your own engine; bind migrate_engine to your metadata | |
|
28 | """ | |
|
29 | _reset_base(migrate_engine) | |
|
30 | from rhodecode.lib.dbmigrate.schema import db_4_7_0_0 as db | |
|
31 | ||
|
32 | # issue fixups | |
|
33 | fixups(db, meta.Session) | |
|
34 | ||
|
35 | ||
|
36 | def downgrade(migrate_engine): | |
|
37 | meta = MetaData() | |
|
38 | meta.bind = migrate_engine | |
|
39 | ||
|
40 | ||
|
41 | def fixups(models, _SESSION): | |
|
42 | ||
|
43 | cleanup_if_present = ( | |
|
44 | models.RhodeCodeUi.HOOK_PRETX_PUSH, | |
|
45 | ) | |
|
46 | ||
|
47 | for hook in cleanup_if_present: | |
|
48 | ui_cfg = models.RhodeCodeUi.query().filter( | |
|
49 | models.RhodeCodeUi.ui_key == hook).scalar() | |
|
50 | if ui_cfg is not None: | |
|
51 | log.info('Removing RhodeCodeUI for hook "%s".', hook) | |
|
52 | _SESSION().delete(ui_cfg) | |
|
53 | ||
|
54 | to_add = [ | |
|
55 | (models.RhodeCodeUi.HOOK_PRETX_PUSH, | |
|
56 | 'python:vcsserver.hooks.pre_push'), | |
|
57 | ] | |
|
58 | ||
|
59 | for hook, value in to_add: | |
|
60 | log.info('Adding RhodeCodeUI for hook "%s".', hook) | |
|
61 | create_or_update_hook(models.RhodeCodeUi, hook, value, _SESSION) | |
|
62 | ||
|
63 | _SESSION().commit() No newline at end of file |
@@ -51,7 +51,7 b' PYRAMID_SETTINGS = {}' | |||
|
51 | 51 | EXTENSIONS = {} |
|
52 | 52 | |
|
53 | 53 | __version__ = ('.'.join((str(each) for each in VERSION[:3]))) |
|
54 |
__dbversion__ = 6 |
|
|
54 | __dbversion__ = 65 # defines current db version for migrations | |
|
55 | 55 | __platform__ = platform.system() |
|
56 | 56 | __license__ = 'AGPLv3, and Commercial License' |
|
57 | 57 | __author__ = 'RhodeCode GmbH' |
@@ -315,6 +315,7 b' class DbManage(object):' | |||
|
315 | 315 | (RhodeCodeUi.HOOK_PRE_PULL, 'python:vcsserver.hooks.pre_pull'), |
|
316 | 316 | (RhodeCodeUi.HOOK_PULL, 'python:vcsserver.hooks.log_pull_action'), |
|
317 | 317 | (RhodeCodeUi.HOOK_PRE_PUSH, 'python:vcsserver.hooks.pre_push'), |
|
318 | (RhodeCodeUi.HOOK_PRETX_PUSH, 'python:vcsserver.hooks.pre_push'), | |
|
318 | 319 | (RhodeCodeUi.HOOK_PUSH, 'python:vcsserver.hooks.log_push_action'), |
|
319 | 320 | |
|
320 | 321 | ] |
@@ -411,6 +411,7 b' def config_data_from_db(clear_session=Tr' | |||
|
411 | 411 | skip_entries.append(('hooks', RhodeCodeUi.HOOK_PRE_PULL)) |
|
412 | 412 | if 'push' not in enabled_hook_classes: |
|
413 | 413 | skip_entries.append(('hooks', RhodeCodeUi.HOOK_PRE_PUSH)) |
|
414 | skip_entries.append(('hooks', RhodeCodeUi.HOOK_PRETX_PUSH)) | |
|
414 | 415 | |
|
415 | 416 | config = [entry for entry in config if entry[:2] not in skip_entries] |
|
416 | 417 |
@@ -350,6 +350,7 b' class RhodeCodeUi(Base, BaseModel):' | |||
|
350 | 350 | HOOK_PRE_PULL = 'preoutgoing.pre_pull' |
|
351 | 351 | HOOK_PULL = 'outgoing.pull_logger' |
|
352 | 352 | HOOK_PRE_PUSH = 'prechangegroup.pre_push' |
|
353 | HOOK_PRETX_PUSH = 'pretxnchangegroup.pre_push' | |
|
353 | 354 | HOOK_PUSH = 'changegroup.push_logger' |
|
354 | 355 | |
|
355 | 356 | # TODO: johbo: Unify way how hooks are configured for git and hg, |
@@ -50,8 +50,8 b' class SettingNotFound(Exception):' | |||
|
50 | 50 | class SettingsModel(BaseModel): |
|
51 | 51 | BUILTIN_HOOKS = ( |
|
52 | 52 | RhodeCodeUi.HOOK_REPO_SIZE, RhodeCodeUi.HOOK_PUSH, |
|
53 |
RhodeCodeUi.HOOK_PRE_PUSH, RhodeCodeUi.HOOK_P |
|
|
54 | RhodeCodeUi.HOOK_PRE_PULL) | |
|
53 | RhodeCodeUi.HOOK_PRE_PUSH, RhodeCodeUi.HOOK_PRETX_PUSH, | |
|
54 | RhodeCodeUi.HOOK_PULL, RhodeCodeUi.HOOK_PRE_PULL) | |
|
55 | 55 | HOOKS_SECTION = 'hooks' |
|
56 | 56 | |
|
57 | 57 | def __init__(self, sa=None, repo=None): |
@@ -81,23 +81,27 b' def disable_hooks(request, hooks):' | |||
|
81 | 81 | |
|
82 | 82 | |
|
83 | 83 | HOOK_PRE_PUSH = db.RhodeCodeUi.HOOK_PRE_PUSH |
|
84 | HOOK_PRETX_PUSH = db.RhodeCodeUi.HOOK_PRETX_PUSH | |
|
84 | 85 | HOOK_PUSH = db.RhodeCodeUi.HOOK_PUSH |
|
85 | 86 | HOOK_PRE_PULL = db.RhodeCodeUi.HOOK_PRE_PULL |
|
86 | 87 | HOOK_PULL = db.RhodeCodeUi.HOOK_PULL |
|
87 | 88 | HOOK_REPO_SIZE = db.RhodeCodeUi.HOOK_REPO_SIZE |
|
88 | 89 | |
|
89 | 90 | HG_HOOKS = frozenset( |
|
90 |
(HOOK_PRE_PULL, HOOK_PULL, HOOK_PRE_PUSH, HOOK_PUSH, |
|
|
91 | (HOOK_PRE_PULL, HOOK_PULL, HOOK_PRE_PUSH, HOOK_PRETX_PUSH, HOOK_PUSH, | |
|
92 | HOOK_REPO_SIZE)) | |
|
91 | 93 | |
|
92 | 94 | |
|
93 | 95 | @pytest.mark.parametrize('disabled_hooks,expected_hooks', [ |
|
94 | 96 | ([], HG_HOOKS), |
|
95 | ([HOOK_PRE_PUSH, HOOK_REPO_SIZE], [HOOK_PRE_PULL, HOOK_PULL, HOOK_PUSH]), | |
|
96 | 97 | (HG_HOOKS, []), |
|
98 | ||
|
99 | ([HOOK_PRE_PUSH, HOOK_PRETX_PUSH, HOOK_REPO_SIZE], [HOOK_PRE_PULL, HOOK_PULL, HOOK_PUSH]), | |
|
100 | ||
|
97 | 101 | # When a pull/push hook is disabled, its pre-pull/push counterpart should |
|
98 | 102 | # be disabled too. |
|
99 | 103 | ([HOOK_PUSH], [HOOK_PRE_PULL, HOOK_PULL, HOOK_REPO_SIZE]), |
|
100 | ([HOOK_PULL], [HOOK_PRE_PUSH, HOOK_PUSH, HOOK_REPO_SIZE]), | |
|
104 | ([HOOK_PULL], [HOOK_PRE_PUSH, HOOK_PRETX_PUSH, HOOK_PUSH, HOOK_REPO_SIZE]), | |
|
101 | 105 | ]) |
|
102 | 106 | def test_make_db_config_hg_hooks(pylonsapp, request, disabled_hooks, |
|
103 | 107 | expected_hooks): |
General Comments 0
You need to be logged in to leave comments.
Login now