Show More
@@ -1,240 +1,242 b'' | |||||
1 | ================================ |
|
1 | ================================ | |
2 | Test corner case around bookmark |
|
2 | Test corner case around bookmark | |
3 | ================================ |
|
3 | ================================ | |
4 |
|
4 | |||
5 | This test file is meant to gather test around bookmark that are specific |
|
5 | This test file is meant to gather test around bookmark that are specific | |
6 | enough to not find a place elsewhere. |
|
6 | enough to not find a place elsewhere. | |
7 |
|
7 | |||
8 | Test bookmark/changelog race condition |
|
8 | Test bookmark/changelog race condition | |
9 | ====================================== |
|
9 | ====================================== | |
10 |
|
10 | |||
11 | The data from the bookmark file are filtered to only contains bookmark with |
|
11 | The data from the bookmark file are filtered to only contains bookmark with | |
12 | node known to the changelog. If the cache invalidation between these two bits |
|
12 | node known to the changelog. If the cache invalidation between these two bits | |
13 | goes wrong, bookmark can be dropped. |
|
13 | goes wrong, bookmark can be dropped. | |
14 |
|
14 | |||
15 | global setup |
|
15 | global setup | |
16 | ------------ |
|
16 | ------------ | |
17 |
|
17 | |||
18 | $ cat >> $HGRCPATH << EOF |
|
18 | $ cat >> $HGRCPATH << EOF | |
19 | > [ui] |
|
19 | > [ui] | |
20 | > ssh = "$PYTHON" "$TESTDIR/dummyssh" |
|
20 | > ssh = "$PYTHON" "$TESTDIR/dummyssh" | |
21 | > [server] |
|
21 | > [server] | |
22 | > concurrent-push-mode=check-related |
|
22 | > concurrent-push-mode=check-related | |
23 | > EOF |
|
23 | > EOF | |
24 |
|
24 | |||
25 | Setup |
|
25 | Setup | |
26 | ----- |
|
26 | ----- | |
27 |
|
27 | |||
28 | initial repository setup |
|
28 | initial repository setup | |
29 |
|
29 | |||
30 | $ hg init bookrace-server |
|
30 | $ hg init bookrace-server | |
31 | $ cd bookrace-server |
|
31 | $ cd bookrace-server | |
32 | $ echo a > a |
|
32 | $ echo a > a | |
33 | $ hg add a |
|
33 | $ hg add a | |
34 | $ hg commit -m root |
|
34 | $ hg commit -m root | |
35 | $ echo a >> a |
|
35 | $ echo a >> a | |
36 | $ hg bookmark book-A |
|
36 | $ hg bookmark book-A | |
37 | $ hg commit -m A0 |
|
37 | $ hg commit -m A0 | |
38 | $ hg up 'desc(root)' |
|
38 | $ hg up 'desc(root)' | |
39 | 1 files updated, 0 files merged, 0 files removed, 0 files unresolved |
|
39 | 1 files updated, 0 files merged, 0 files removed, 0 files unresolved | |
40 | (leaving bookmark book-A) |
|
40 | (leaving bookmark book-A) | |
41 | $ echo b > b |
|
41 | $ echo b > b | |
42 | $ hg add b |
|
42 | $ hg add b | |
43 | $ hg bookmark book-B |
|
43 | $ hg bookmark book-B | |
44 | $ hg commit -m B0 |
|
44 | $ hg commit -m B0 | |
45 | created new head |
|
45 | created new head | |
46 | $ hg up null |
|
46 | $ hg up null | |
47 | 0 files updated, 0 files merged, 2 files removed, 0 files unresolved |
|
47 | 0 files updated, 0 files merged, 2 files removed, 0 files unresolved | |
48 | (leaving bookmark book-B) |
|
48 | (leaving bookmark book-B) | |
49 | $ hg phase --public --rev 'all()' |
|
49 | $ hg phase --public --rev 'all()' | |
50 | $ hg log -G |
|
50 | $ hg log -G | |
51 | o changeset: 2:c79985706978 |
|
51 | o changeset: 2:c79985706978 | |
52 | | bookmark: book-B |
|
52 | | bookmark: book-B | |
53 | | tag: tip |
|
53 | | tag: tip | |
54 | | parent: 0:6569b5a81c7e |
|
54 | | parent: 0:6569b5a81c7e | |
55 | | user: test |
|
55 | | user: test | |
56 | | date: Thu Jan 01 00:00:00 1970 +0000 |
|
56 | | date: Thu Jan 01 00:00:00 1970 +0000 | |
57 | | summary: B0 |
|
57 | | summary: B0 | |
58 | | |
|
58 | | | |
59 | | o changeset: 1:39c28d785860 |
|
59 | | o changeset: 1:39c28d785860 | |
60 | |/ bookmark: book-A |
|
60 | |/ bookmark: book-A | |
61 | | user: test |
|
61 | | user: test | |
62 | | date: Thu Jan 01 00:00:00 1970 +0000 |
|
62 | | date: Thu Jan 01 00:00:00 1970 +0000 | |
63 | | summary: A0 |
|
63 | | summary: A0 | |
64 | | |
|
64 | | | |
65 | o changeset: 0:6569b5a81c7e |
|
65 | o changeset: 0:6569b5a81c7e | |
66 | user: test |
|
66 | user: test | |
67 | date: Thu Jan 01 00:00:00 1970 +0000 |
|
67 | date: Thu Jan 01 00:00:00 1970 +0000 | |
68 | summary: root |
|
68 | summary: root | |
69 |
|
69 | |||
70 | $ hg book |
|
70 | $ hg book | |
71 | book-A 1:39c28d785860 |
|
71 | book-A 1:39c28d785860 | |
72 | book-B 2:c79985706978 |
|
72 | book-B 2:c79985706978 | |
73 | $ cd .. |
|
73 | $ cd .. | |
74 |
|
74 | |||
75 | Add new changeset on each bookmark in distinct clones |
|
75 | Add new changeset on each bookmark in distinct clones | |
76 |
|
76 | |||
77 | $ hg clone ssh://user@dummy/bookrace-server client-A |
|
77 | $ hg clone ssh://user@dummy/bookrace-server client-A | |
78 | requesting all changes |
|
78 | requesting all changes | |
79 | adding changesets |
|
79 | adding changesets | |
80 | adding manifests |
|
80 | adding manifests | |
81 | adding file changes |
|
81 | adding file changes | |
82 | added 3 changesets with 3 changes to 2 files (+1 heads) |
|
82 | added 3 changesets with 3 changes to 2 files (+1 heads) | |
83 | new changesets 6569b5a81c7e:c79985706978 |
|
83 | new changesets 6569b5a81c7e:c79985706978 | |
84 | updating to branch default |
|
84 | updating to branch default | |
85 | 2 files updated, 0 files merged, 0 files removed, 0 files unresolved |
|
85 | 2 files updated, 0 files merged, 0 files removed, 0 files unresolved | |
86 | $ hg -R client-A update book-A |
|
86 | $ hg -R client-A update book-A | |
87 | 1 files updated, 0 files merged, 1 files removed, 0 files unresolved |
|
87 | 1 files updated, 0 files merged, 1 files removed, 0 files unresolved | |
88 | (activating bookmark book-A) |
|
88 | (activating bookmark book-A) | |
89 | $ echo a >> client-A/a |
|
89 | $ echo a >> client-A/a | |
90 | $ hg -R client-A commit -m A1 |
|
90 | $ hg -R client-A commit -m A1 | |
91 | $ hg clone ssh://user@dummy/bookrace-server client-B |
|
91 | $ hg clone ssh://user@dummy/bookrace-server client-B | |
92 | requesting all changes |
|
92 | requesting all changes | |
93 | adding changesets |
|
93 | adding changesets | |
94 | adding manifests |
|
94 | adding manifests | |
95 | adding file changes |
|
95 | adding file changes | |
96 | added 3 changesets with 3 changes to 2 files (+1 heads) |
|
96 | added 3 changesets with 3 changes to 2 files (+1 heads) | |
97 | new changesets 6569b5a81c7e:c79985706978 |
|
97 | new changesets 6569b5a81c7e:c79985706978 | |
98 | updating to branch default |
|
98 | updating to branch default | |
99 | 2 files updated, 0 files merged, 0 files removed, 0 files unresolved |
|
99 | 2 files updated, 0 files merged, 0 files removed, 0 files unresolved | |
100 | $ hg -R client-B update book-B |
|
100 | $ hg -R client-B update book-B | |
101 | 0 files updated, 0 files merged, 0 files removed, 0 files unresolved |
|
101 | 0 files updated, 0 files merged, 0 files removed, 0 files unresolved | |
102 | (activating bookmark book-B) |
|
102 | (activating bookmark book-B) | |
103 | $ echo b >> client-B/b |
|
103 | $ echo b >> client-B/b | |
104 | $ hg -R client-B commit -m B1 |
|
104 | $ hg -R client-B commit -m B1 | |
105 |
|
105 | |||
106 | extension to reproduce the race |
|
106 | extension to reproduce the race | |
107 | ------------------------------- |
|
107 | ------------------------------- | |
108 |
|
108 | |||
109 | If two process are pushing we want to make sure the following happens: |
|
109 | If two process are pushing we want to make sure the following happens: | |
110 |
|
110 | |||
111 | * process A read changelog |
|
111 | * process A read changelog | |
112 | * process B to its full push |
|
112 | * process B to its full push | |
113 | * process A read bookmarks |
|
113 | * process A read bookmarks | |
114 | * process A proceed with rest of the push |
|
114 | * process A proceed with rest of the push | |
115 |
|
115 | |||
116 | We build a server side extension for this purpose |
|
116 | We build a server side extension for this purpose | |
117 |
|
117 | |||
118 | $ cat > bookrace.py << EOF |
|
118 | $ cat > bookrace.py << EOF | |
119 | > import os |
|
119 | > import os | |
120 | > import time |
|
120 | > import time | |
121 | > import atexit |
|
121 | > import atexit | |
122 | > from mercurial import error, extensions, bookmarks |
|
122 | > from mercurial import error, extensions, bookmarks | |
123 | > |
|
123 | > | |
124 | > def wait(): |
|
124 | > def wait(repo): | |
125 | > if not os.path.exists('push-A-started'): |
|
125 | > if not os.path.exists('push-A-started'): | |
|
126 | > assert repo._currentlock(repo._lockref) is None | |||
|
127 | > assert repo._currentlock(repo._wlockref) is None | |||
126 | > print('setting raced push up') |
|
128 | > print('setting raced push up') | |
127 | > with open('push-A-started', 'w'): |
|
129 | > with open('push-A-started', 'w'): | |
128 | > pass |
|
130 | > pass | |
129 | > clock = 300 |
|
131 | > clock = 300 | |
130 | > while not os.path.exists('push-B-done'): |
|
132 | > while not os.path.exists('push-B-done'): | |
131 | > clock -= 1 |
|
133 | > clock -= 1 | |
132 | > if clock <= 0: |
|
134 | > if clock <= 0: | |
133 | > raise error.Abort("race scenario timed out") |
|
135 | > raise error.Abort("race scenario timed out") | |
134 | > time.sleep(0.1) |
|
136 | > time.sleep(0.1) | |
135 | > |
|
137 | > | |
136 | > def wrapinit(orig, self, repo): |
|
138 | > def wrapinit(orig, self, repo): | |
137 | > wait() |
|
139 | > wait(repo) | |
138 | > return orig(self, repo) |
|
140 | > return orig(self, repo) | |
139 | > def uisetup(ui): |
|
141 | > def uisetup(ui): | |
140 | > extensions.wrapfunction(bookmarks.bmstore, '__init__', wrapinit) |
|
142 | > extensions.wrapfunction(bookmarks.bmstore, '__init__', wrapinit) | |
141 | > def e(): |
|
143 | > def e(): | |
142 | > with open('push-A-done', 'w'): |
|
144 | > with open('push-A-done', 'w'): | |
143 | > pass |
|
145 | > pass | |
144 | > atexit.register(e) |
|
146 | > atexit.register(e) | |
145 | > EOF |
|
147 | > EOF | |
146 |
|
148 | |||
147 | Actual test |
|
149 | Actual test | |
148 | ----------- |
|
150 | ----------- | |
149 |
|
151 | |||
150 | Start the raced push. |
|
152 | Start the raced push. | |
151 |
|
153 | |||
152 | $ cat >> bookrace-server/.hg/hgrc << EOF |
|
154 | $ cat >> bookrace-server/.hg/hgrc << EOF | |
153 | > [extensions] |
|
155 | > [extensions] | |
154 | > bookrace=$TESTTMP/bookrace.py |
|
156 | > bookrace=$TESTTMP/bookrace.py | |
155 | > EOF |
|
157 | > EOF | |
156 | $ hg push -R client-A -r book-A >push-output.txt 2>&1 & |
|
158 | $ hg push -R client-A -r book-A >push-output.txt 2>&1 & | |
157 |
|
159 | |||
158 | Wait up to 30 seconds for that push to start. |
|
160 | Wait up to 30 seconds for that push to start. | |
159 |
|
161 | |||
160 | $ clock=30 |
|
162 | $ clock=30 | |
161 | $ while [ ! -f push-A-started ] && [ $clock -gt 0 ] ; do |
|
163 | $ while [ ! -f push-A-started ] && [ $clock -gt 0 ] ; do | |
162 | > clock=`expr $clock - 1` |
|
164 | > clock=`expr $clock - 1` | |
163 | > sleep 1 |
|
165 | > sleep 1 | |
164 | > done |
|
166 | > done | |
165 |
|
167 | |||
166 | Do the other push. |
|
168 | Do the other push. | |
167 |
|
169 | |||
168 | $ cat >> bookrace-server/.hg/hgrc << EOF |
|
170 | $ cat >> bookrace-server/.hg/hgrc << EOF | |
169 | > [extensions] |
|
171 | > [extensions] | |
170 | > bookrace=! |
|
172 | > bookrace=! | |
171 | > EOF |
|
173 | > EOF | |
172 |
|
174 | |||
173 | $ hg push -R client-B -r book-B |
|
175 | $ hg push -R client-B -r book-B | |
174 | pushing to ssh://user@dummy/bookrace-server |
|
176 | pushing to ssh://user@dummy/bookrace-server | |
175 | searching for changes |
|
177 | searching for changes | |
176 | remote: adding changesets |
|
178 | remote: adding changesets | |
177 | remote: adding manifests |
|
179 | remote: adding manifests | |
178 | remote: adding file changes |
|
180 | remote: adding file changes | |
179 | remote: added 1 changesets with 1 changes to 1 files |
|
181 | remote: added 1 changesets with 1 changes to 1 files | |
180 | updating bookmark book-B |
|
182 | updating bookmark book-B | |
181 |
|
183 | |||
182 | Signal the raced put that we are done (it waits up to 30 seconds). |
|
184 | Signal the raced put that we are done (it waits up to 30 seconds). | |
183 |
|
185 | |||
184 | $ touch push-B-done |
|
186 | $ touch push-B-done | |
185 |
|
187 | |||
186 | Wait for the raced push to finish (with the remaning of the initial 30 seconds). |
|
188 | Wait for the raced push to finish (with the remaning of the initial 30 seconds). | |
187 |
|
189 | |||
188 | $ while [ ! -f push-A-done ] && [ $clock -gt 0 ] ; do |
|
190 | $ while [ ! -f push-A-done ] && [ $clock -gt 0 ] ; do | |
189 | > clock=`expr $clock - 1` |
|
191 | > clock=`expr $clock - 1` | |
190 | > sleep 1 |
|
192 | > sleep 1 | |
191 | > done |
|
193 | > done | |
192 |
|
194 | |||
193 | Check raced push output. |
|
195 | Check raced push output. | |
194 |
|
196 | |||
195 | $ cat push-output.txt |
|
197 | $ cat push-output.txt | |
196 | pushing to ssh://user@dummy/bookrace-server |
|
198 | pushing to ssh://user@dummy/bookrace-server | |
197 | searching for changes |
|
199 | searching for changes | |
198 | remote: setting raced push up |
|
200 | remote: setting raced push up | |
199 | remote: adding changesets |
|
201 | remote: adding changesets | |
200 | remote: adding manifests |
|
202 | remote: adding manifests | |
201 | remote: adding file changes |
|
203 | remote: adding file changes | |
202 | remote: added 1 changesets with 1 changes to 1 files |
|
204 | remote: added 1 changesets with 1 changes to 1 files | |
203 | updating bookmark book-A |
|
205 | updating bookmark book-A | |
204 |
|
206 | |||
205 | Check result of the push. |
|
207 | Check result of the push. | |
206 |
|
208 | |||
207 | $ hg -R bookrace-server log -G |
|
209 | $ hg -R bookrace-server log -G | |
208 | o changeset: 4:9ce3b28c16de |
|
210 | o changeset: 4:9ce3b28c16de | |
209 | | bookmark: book-A |
|
211 | | bookmark: book-A | |
210 | | tag: tip |
|
212 | | tag: tip | |
211 | | parent: 1:39c28d785860 |
|
213 | | parent: 1:39c28d785860 | |
212 | | user: test |
|
214 | | user: test | |
213 | | date: Thu Jan 01 00:00:00 1970 +0000 |
|
215 | | date: Thu Jan 01 00:00:00 1970 +0000 | |
214 | | summary: A1 |
|
216 | | summary: A1 | |
215 | | |
|
217 | | | |
216 | | o changeset: 3:f26c3b5167d1 |
|
218 | | o changeset: 3:f26c3b5167d1 | |
217 | | | bookmark: book-B |
|
219 | | | bookmark: book-B | |
218 | | | user: test |
|
220 | | | user: test | |
219 | | | date: Thu Jan 01 00:00:00 1970 +0000 |
|
221 | | | date: Thu Jan 01 00:00:00 1970 +0000 | |
220 | | | summary: B1 |
|
222 | | | summary: B1 | |
221 | | | |
|
223 | | | | |
222 | | o changeset: 2:c79985706978 |
|
224 | | o changeset: 2:c79985706978 | |
223 | | | parent: 0:6569b5a81c7e |
|
225 | | | parent: 0:6569b5a81c7e | |
224 | | | user: test |
|
226 | | | user: test | |
225 | | | date: Thu Jan 01 00:00:00 1970 +0000 |
|
227 | | | date: Thu Jan 01 00:00:00 1970 +0000 | |
226 | | | summary: B0 |
|
228 | | | summary: B0 | |
227 | | | |
|
229 | | | | |
228 | o | changeset: 1:39c28d785860 |
|
230 | o | changeset: 1:39c28d785860 | |
229 | |/ user: test |
|
231 | |/ user: test | |
230 | | date: Thu Jan 01 00:00:00 1970 +0000 |
|
232 | | date: Thu Jan 01 00:00:00 1970 +0000 | |
231 | | summary: A0 |
|
233 | | summary: A0 | |
232 | | |
|
234 | | | |
233 | o changeset: 0:6569b5a81c7e |
|
235 | o changeset: 0:6569b5a81c7e | |
234 | user: test |
|
236 | user: test | |
235 | date: Thu Jan 01 00:00:00 1970 +0000 |
|
237 | date: Thu Jan 01 00:00:00 1970 +0000 | |
236 | summary: root |
|
238 | summary: root | |
237 |
|
239 | |||
238 | $ hg -R bookrace-server book |
|
240 | $ hg -R bookrace-server book | |
239 | book-A 4:9ce3b28c16de |
|
241 | book-A 4:9ce3b28c16de | |
240 | book-B 3:f26c3b5167d1 |
|
242 | book-B 3:f26c3b5167d1 |
General Comments 0
You need to be logged in to leave comments.
Login now