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

Fix possible delete files race condition in gdpr_SUITE #2295

Merged
merged 1 commit into from May 8, 2019

Conversation

aleklisi
Copy link
Contributor

@aleklisi aleklisi commented May 8, 2019

No description provided.

@mongoose-im
Copy link
Collaborator

mongoose-im commented May 8, 2019

6421.1 / Erlang 19.3 / small_tests / b65b835
Reports root / small


6421.2 / Erlang 19.3 / internal_mnesia / b65b835
Reports root/ big
OK: 1211 / Failed: 0 / User-skipped: 65 / Auto-skipped: 0


6421.6 / Erlang 19.3 / elasticsearch_and_cassandra_mnesia / b65b835
Reports root/ big
OK: 469 / Failed: 0 / User-skipped: 8 / Auto-skipped: 0


6421.3 / Erlang 19.3 / mysql_redis / b65b835
Reports root/ big
OK: 3082 / Failed: 0 / User-skipped: 232 / Auto-skipped: 0


6421.5 / Erlang 19.3 / ldap_mnesia / b65b835
Reports root/ big
OK: 1175 / Failed: 0 / User-skipped: 101 / Auto-skipped: 0


6421.4 / Erlang 19.3 / odbc_mssql_mnesia / b65b835
Reports root/ big
OK: 3094 / Failed: 2 / User-skipped: 230 / Auto-skipped: 0

pubsub_SUITE:tree+hometree_specific:can_create_node_with_existing_parent_path
{error,
  {timeout_when_waiting_for_stanza,
    [{escalus_client,wait_for_stanza,
       [{client,<<"alicE70.439387@localhost/res1">>,escalus_tcp,
          <0.11658.3>,
          [{event_manager,<0.11632.3>},
           {server,<<"localhost">>},
           {username,<<"alicE70.439387">>},
           {resource,<<"res1">>}],
          [{event_client,
             [{event_manager,<0.11632.3>},
            {server,<<"localhost">>},
            {username,<<"alicE70.439387">>},
            {resource,<<"res1">>}]},
           {resource,<<"res1">>},
           {username,<<"alicE70.439387">>},
           {server,<<"localhost">>},
           {host,<<"localhost">>},
           {port,5222},
           {auth,{escalus_auth,auth_plain}},
           {wspath,undefined},
           {username,<<"alicE70.439387">>},
           {server,<<"localhost">>},
           {password,<<"matygrysa">>},
           {stream_id,<<"3445503A27AD51EE">>}]},
        5000],
       [{file,
          "/home/travis/build/esl/MongooseIM/big_tests/_build/default/lib/escalus/src/escalus_client.erl"},
        {line,138}]},
     {pubsub_tools,receive_response,3,
       [{file,"pubsub_tools.erl"},{line,457}]},
     {pubsub_tools,receive_and_check_response,4,
       [{file,"pubsub_tools.erl"},{line,448}]},
     {escalus_story,story,4,
       [{file,
          "/home/travis/build/esl/MongooseIM/big_tests/_build/default/lib/escalus/src/escalus_story.erl"},
        {line,72}]},
     {test_server,ts_tc,3,[{file,"test_server.erl"...

Report log

sm_SUITE:parallel_manual_ack_freq_1:resume_session_state_stop_c2s
{error,{{assertion_failed,assert_many,true,
              [is_presence,#Fun<sm_SUITE.16.86460923>],
              [{xmlel,<<"presence">>,
                  [{<<"from">>,
                  <<"alicE73.848876@localhost/escalus-default-resource">>},
                   {<<"to">>,
                  <<"alice73.848876@localhost/escalus-default-resource">>},
                   {<<"xml:lang">>,<<"en">>}],
                  []},
               {xmlel,<<"presence">>,
                  [{<<"from">>,
                  <<"alicE73.848876@localhost/escalus-default-resource">>},
                   {<<"to">>,
                  <<"alice73.848876@localhost/escalus-default-resource">>},
                   {<<"xml:lang">>,<<"en">>}],
                  []}],
              "   <presence from='alicE73.848876@localhost/escalus-default-resource' to='alice73.848876@localhost/escalus-default-resource' xml:lang='en'/>   <presence from='alicE73.848876@localhost/escalus-default-resource' to='alice73.848876@localhost/escalus-default-resource' xml:lang='en'/>"},
    [{escalus_new_assert,assert_true,2,
               [{file,"/home/travis/build/esl/MongooseIM/big_tests/_build/default/lib/escalus/src/escalus_new_assert.erl"},
                {line,84}]},
     {sm_SUITE,resume_session_state_stop_c2s,1,
           [{file,"sm_SUITE.erl"},{line,581}]},
     {test_server,ts_tc,3,[{file,"test_server.erl"},{line,1529}]},
     {test_server,run_test_case_eval1,6,
            [{file,"test_server.erl"},{line,1045}]},
     {...

Report log


6421.8 / Erlang 20.0 / pgsql_mnesia / b65b835
Reports root/ big / small
OK: 3116 / Failed: 0 / User-skipped: 198 / Auto-skipped: 0


6421.9 / Erlang 21.0 / riak_mnesia / b65b835
Reports root/ big / small
OK: 1446 / Failed: 0 / User-skipped: 63 / Auto-skipped: 0

@aleklisi
Copy link
Contributor Author

aleklisi commented May 8, 2019

The failing tests looks unrelated

@fenek fenek merged commit 0b85d95 into gdpr-retrieve-clean May 8, 2019
@fenek fenek deleted the Fix-possible-delete-files-race-conditon branch May 8, 2019 11:52
@fenek fenek added this to the MongooseIM 3.3.0++ milestone Jun 18, 2019
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.

None yet

3 participants