improvement: deprecate openldap dependency for apisix

[Zhenye-Na]

This PR is in the scope of this issue in apisix repo: apache/apisix#7865

[CLAassistant]

All committers have signed the CLA.

[monkeyDluffy6017]

Please make the ci pass

[Zhenye-Na]

Please make the ci pass

Please correct me if I am wrong but it seems like a circular dependency happened between #337 and apache/apisix#10176

1. This PR pulls latest of apisix where openldap is in the rockspec
2. fix: remove openldap dependencies from apisix apache/apisix#10176 depends on the build tool change in this PR

[kingluo]

@Zhenye-Na Please fix the CI errors first.

[Zhenye-Na]

As I mentioend above, the circular dependency of this repo and apache/apisix occurs here: https://github.com/api7/apisix-build-tools/actions/runs/6445185009/workflow?pr=337#L44

this package is building apisix with the master branch, where lualdap/openldap exists. However, this PR delted the dependency which caused the issue.

If it is ok to you @kingluo , I could redirect the line above to my own forked branch to see if CI could pass.

[kingluo]

@Zhenye-Na
ok, if you confirm these 4 errors are expected due to circular dependencies (I think so after reading the code), I'll approve it.
@monkeyDluffy6017 @membphis @moonming
What about your opinions?

[Zhenye-Na]

@Zhenye-Na ok, if you confirm these 4 errors are expected due to circular dependencies (I think so after reading the code), I'll approve it. @monkeyDluffy6017 @membphis @moonming What about your opinions?

Please let me know what else I could test to make sure there is nothing I missed regarding this dependency deprecation. Meanwhile, if there is a way to inject my personal fork to test things out here, that will be even better.

Thanks!

[Revolyssup]

@Zhenye-Na Is this PR prerequisite for apache/apisix#10176?

[Revolyssup]

@Zhenye-Na Can you fix the merge conflicts?

[Zhenye-Na]

@Zhenye-Na Is this PR prerequisite for apache/apisix#10176?

Yes

[Zhenye-Na]

@Zhenye-Na Can you fix the merge conflicts?

I could work on this, but will definitely appreciate any guidance of merging this PR later

[Revolyssup]

@Zhenye-Na Can you fix the merge conflicts?

I could work on this, but will definitely appreciate any guidance of merging this PR later

When can you work on this?

[Zhenye-Na]

@Zhenye-Na Can you fix the merge conflicts?

I could work on this, but will definitely appreciate any guidance of merging this PR later

When can you work on this?

Please allow me to rebase and resolve any merge conflicts by end of this Friday.

I could ping you here once I got next revision out, if it works for you ?

thanks

[Zhenye-Na]

Missed the committed SLA, will prioritize this patch during this work week.

Thanks

[Zhenye-Na]

@Revolyssup just resolved merge conflicts